saveJSON test update

Author: ZeeCoder

CHANGELOG.md

@@ -11,7 +11,7 @@ and this project adheres to [Semantic Versioning](http://semver.org/spec/v2.0.0.
 
 ### Fixed
 
-* Containers can now be safely nested.
+* Containers can now be safely nested. (Container Boundaries)
 
 ## [2.2.1]
 

packages/postcss-container-query/src/containerQuery.js

@@ -36,7 +36,6 @@ function extractContainerUnits(node, isContainer = false) {
 }
 
 /**
- * @todo refactor this to use a builder, which'll make testing easier too
  * @param {{
  *   getJSON: function,
  *   singleContainer: boolean,

packages/postcss-container-query/src/saveJSON.spec.js

@@ -41,3 +41,26 @@ test("should not write contents when file already with same contents", () => {
       expect(fs.writeFile).toHaveBeenCalledTimes(0);
     });
 });
+
+test("should log error in case there's an issue writing the file", () => {
+  const fs = require("fs");
+  const cssFilePath = __dirname + "/tmp/saveJSON.test.css";
+  const jsonFilePath = `${cssFilePath}.json`;
+  const json = { some: "JSON" };
+
+  fs.readFile = jest.fn((filepath, type, cb) => cb(new Error("error")));
+  fs.writeFile = jest.fn((filepath, content, cb) => cb(new Error("error")));
+  console.error = jest.fn();
+
+  expect.hasAssertions();
+
+  return Promise.resolve()
+    .then(() => saveJSON(cssFilePath, json))
+    .then(() => {
+      expect(fs.readFile).toHaveBeenCalledTimes(1);
+      expect(fs.writeFile).toHaveBeenCalledTimes(1);
+      expect(fs.writeFile.mock.calls[0][0]).toBe(jsonFilePath);
+      expect(fs.writeFile.mock.calls[0][1]).toBe(JSON.stringify(json));
+      expect(console.error).toHaveBeenCalled();
+    });
+});