Skip to content

Commit d6aa49c

Browse files
committed
Fix unsanitized JSON snapshot
1 parent 5b03310 commit d6aa49c

File tree

2 files changed

+9
-4
lines changed

2 files changed

+9
-4
lines changed

test/jest/__snapshots__/analyze.test.js.snap

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,10 +1,13 @@
11
// Jest Snapshot v1, https://goo.gl/fbAQLP
22

33
exports[`analyze in ./analyze/dsl handles json option 1`] = `
4-
"
5-
{"warnings":[{"range":{"start":{"line":25,"col":0,"pos":853},"end":{"line":25,"col":35,"pos":888}},"filename":"/home/naver/git/eask-cli/test/jest/analyze/dsl/Eask","message":"💡 Pkg-file seems to be missing \`check-pkg.el'"},{"range":{"start":{"line":20,"col":0,"pos":709},"end":{"line":20,"col":17,"pos":726}},"filename":"/home/naver/git/eask-cli/test/jest/analyze/dsl/Eask","message":"💡 Warning regarding duplicate license name, GPLv3"},{"range":{"start":{"line":17,"col":0,"pos":613},"end":{"line":17,"col":15,"pos":628}},"filename":"/home/naver/git/eask-cli/test/jest/analyze/dsl/Eask","message":"💡 Warning regarding duplicate author name, name"}],"errors":[{"range":{"start":{"line":46,"col":0,"pos":1696},"end":{"line":50,"col":2,"pos":1908}},"filename":"/home/naver/git/eask-cli/test/jest/analyze/dsl/Eask","message":"✗ Define dependencies with the same name \`f' with different version"},{"range":{"start":{"line":46,"col":0,"pos":1696},"end":{"line":50,"col":2,"pos":1908}},"filename":"/home/naver/git/eask-cli/test/jest/analyze/dsl/Eask","message":"✗ Define dependencies with the same name \`f'"},{"range":{"start":{"line":44,"col":0,"pos":1596},"end":{"line":44,"col":27,"pos":1623}},"filename":"/home/naver/git/eask-cli/test/jest/analyze/dsl/Eask","message":"✗ Define dependencies with the same name \`dash' with different version"},{"range":{"start":{"line":43,"col":0,"pos":1576},"end":{"line":43,"col":19,"pos":1595}},"filename":"/home/naver/git/eask-cli/test/jest/analyze/dsl/Eask","message":"✗ Define dependencies with the same name \`dash'"},{"range":{"start":{"line":40,"col":0,"pos":1474},"end":{"line":40,"col":20,"pos":1494}},"filename":"/home/naver/git/eask-cli/test/jest/analyze/dsl/Eask","message":"✗ Define dependencies with the same name \`emacs'"},{"range":{"start":{"line":37,"col":0,"pos":1304},"end":{"line":37,"col":15,"pos":1319}},"filename":"/home/naver/git/eask-cli/test/jest/analyze/dsl/Eask","message":"✗ Unknown package archive \`local'"},{"range":{"start":{"line":37,"col":0,"pos":1304},"end":{"line":37,"col":15,"pos":1319}},"filename":"/home/naver/git/eask-cli/test/jest/analyze/dsl/Eask","message":"✗ Invalid archive name \`local'"},{"range":{"start":{"line":35,"col":0,"pos":1229},"end":{"line":35,"col":24,"pos":1253}},"filename":"/home/naver/git/eask-cli/test/jest/analyze/dsl/Eask","message":"✗ Unknown package archive \`magic-archive'"},{"range":{"start":{"line":33,"col":0,"pos":1214},"end":{"line":33,"col":13,"pos":1227}},"filename":"/home/naver/git/eask-cli/test/jest/analyze/dsl/Eask","message":"✗ Multiple definition of source \`gnu'"},{"range":{"start":{"line":30,"col":0,"pos":1073},"end":{"line":30,"col":61,"pos":1134}},"filename":"/home/naver/git/eask-cli/test/jest/analyze/dsl/Eask","message":"✗ Run-script with the same key name is not allowed: \`test\`"},{"range":{"start":{"line":26,"col":0,"pos":953},"end":{"line":26,"col":35,"pos":988}},"filename":"/home/naver/git/eask-cli/test/jest/analyze/dsl/Eask","message":"✗ Multiple definition of \`package-descriptor'"},{"range":{"start":{"line":23,"col":0,"pos":822},"end":{"line":23,"col":29,"pos":851}},"filename":"/home/naver/git/eask-cli/test/jest/analyze/dsl/Eask","message":"✗ Multiple definition of \`package-file'"},{"range":{"start":{"line":14,"col":0,"pos":517},"end":{"line":14,"col":16,"pos":533}},"filename":"/home/naver/git/eask-cli/test/jest/analyze/dsl/Eask","message":"✗ Multiple definition of \`keywords'"},{"range":{"start":{"line":12,"col":0,"pos":383},"end":{"line":12,"col":55,"pos":438}},"filename":"/home/naver/git/eask-cli/test/jest/analyze/dsl/Eask","message":"✗ Multiple definition of \`website-url'"},{"range":{"start":{"line":9,"col":0,"pos":290},"end":{"line":9,"col":18,"pos":308}},"filename":"/home/naver/git/eask-cli/test/jest/analyze/dsl/Eask","message":"✗ Multiple definition of \`package'"}]}
4+
{
5+
"stderr": "
6+
{"warnings":[{"range":{"start":{"line":25,"col":0,"pos":853},"end":{"line":25,"col":35,"pos":888}},"filename":"~/Eask","message":"💡 Pkg-file seems to be missing \`check-pkg.el'"},{"range":{"start":{"line":20,"col":0,"pos":709},"end":{"line":20,"col":17,"pos":726}},"filename":"~/Eask","message":"💡 Warning regarding duplicate license name, GPLv3"},{"range":{"start":{"line":17,"col":0,"pos":613},"end":{"line":17,"col":15,"pos":628}},"filename":"~/Eask","message":"💡 Warning regarding duplicate author name, name"}],"errors":[{"range":{"start":{"line":46,"col":0,"pos":1696},"end":{"line":50,"col":2,"pos":1908}},"filename":"~/Eask","message":"✗ Define dependencies with the same name \`f' with different version"},{"range":{"start":{"line":46,"col":0,"pos":1696},"end":{"line":50,"col":2,"pos":1908}},"filename":"~/Eask","message":"✗ Define dependencies with the same name \`f'"},{"range":{"start":{"line":44,"col":0,"pos":1596},"end":{"line":44,"col":27,"pos":1623}},"filename":"~/Eask","message":"✗ Define dependencies with the same name \`dash' with different version"},{"range":{"start":{"line":43,"col":0,"pos":1576},"end":{"line":43,"col":19,"pos":1595}},"filename":"~/Eask","message":"✗ Define dependencies with the same name \`dash'"},{"range":{"start":{"line":40,"col":0,"pos":1474},"end":{"line":40,"col":20,"pos":1494}},"filename":"~/Eask","message":"✗ Define dependencies with the same name \`emacs'"},{"range":{"start":{"line":37,"col":0,"pos":1304},"end":{"line":37,"col":15,"pos":1319}},"filename":"~/Eask","message":"✗ Unknown package archive \`local'"},{"range":{"start":{"line":37,"col":0,"pos":1304},"end":{"line":37,"col":15,"pos":1319}},"filename":"~/Eask","message":"✗ Invalid archive name \`local'"},{"range":{"start":{"line":35,"col":0,"pos":1229},"end":{"line":35,"col":24,"pos":1253}},"filename":"~/Eask","message":"✗ Unknown package archive \`magic-archive'"},{"range":{"start":{"line":33,"col":0,"pos":1214},"end":{"line":33,"col":13,"pos":1227}},"filename":"~/Eask","message":"✗ Multiple definition of source \`gnu'"},{"range":{"start":{"line":30,"col":0,"pos":1073},"end":{"line":30,"col":61,"pos":1134}},"filename":"~/Eask","message":"✗ Run-script with the same key name is not allowed: \`test\`"},{"range":{"start":{"line":26,"col":0,"pos":953},"end":{"line":26,"col":35,"pos":988}},"filename":"~/Eask","message":"✗ Multiple definition of \`package-descriptor'"},{"range":{"start":{"line":23,"col":0,"pos":822},"end":{"line":23,"col":29,"pos":851}},"filename":"~/Eask","message":"✗ Multiple definition of \`package-file'"},{"range":{"start":{"line":14,"col":0,"pos":517},"end":{"line":14,"col":16,"pos":533}},"filename":"~/Eask","message":"✗ Multiple definition of \`keywords'"},{"range":{"start":{"line":12,"col":0,"pos":383},"end":{"line":12,"col":55,"pos":438}},"filename":"~/Eask","message":"✗ Multiple definition of \`website-url'"},{"range":{"start":{"line":9,"col":0,"pos":290},"end":{"line":9,"col":18,"pos":308}},"filename":"~/Eask","message":"✗ Multiple definition of \`package'"}]}
67
(Checked 1 file)
7-
"
8+
",
9+
"stdout": "",
10+
}
811
`;
912

1013
exports[`analyze in ./analyze/dsl matches snapshot 1`] = `

test/jest/analyze.test.js

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -40,7 +40,9 @@ describe("analyze", () => {
4040
await ctx.runEask("analyze --json");
4141
} catch (e) {
4242
tryJSON(e.stderr);
43-
expect(e.stderr).toMatchSnapshot();
43+
const res = ctx.errorToCommandOutput(e);
44+
const resClean = res.sanitized().raw();
45+
expect(resClean).toMatchSnapshot();
4446
}
4547
});
4648

0 commit comments

Comments
 (0)