Skip to content

Commit c57239d

Browse files
QUA-218: Use streams to stringify large jsons (#531)
* Use streams to stringify large jsons * QUA-218: Convert some integration tests to system tests Co-authored-by: Federico Moyá <[email protected]>
1 parent 4b8564a commit c57239d

28 files changed

+89
-484
lines changed

integration/eslint_test.js

-37
Original file line numberDiff line numberDiff line change
@@ -29,19 +29,6 @@ describe("eslint integration", function() {
2929
console.error = STDERR
3030
})
3131

32-
describe("eslintrc files with not supported plugins in it", function() {
33-
it("does not raise any error", function() {
34-
this.timeout(3000)
35-
36-
function executeUnsupportedPlugins() {
37-
executeConfig("with_unsupported_plugins/config.json")
38-
}
39-
40-
expect(executeUnsupportedPlugins).to.not.throw()
41-
expect(consoleMock.output).to.not.be.empty
42-
})
43-
})
44-
4532
describe("validating config", function() {
4633
it("raise on file not found", function() {
4734
function executeNoLintrc() {
@@ -70,19 +57,6 @@ describe("eslint integration", function() {
7057
})
7158
})
7259

73-
describe("extends plugin", function() {
74-
it("loads the plugin and does not include repeated issues of not found rules", function() {
75-
this.timeout(8000)
76-
77-
executeConfig("extends_airbnb/config.json")
78-
79-
const ruleDefinitionIssues = consoleMock.output.filter(function(o) {
80-
return o.includes("Definition for rule")
81-
})
82-
expect(ruleDefinitionIssues).to.be.empty
83-
})
84-
})
85-
8660
describe("sanitization", function() {
8761
function withMinifiedSource(config, cb, done) {
8862
temp.mkdir("code", function(err, directory) {
@@ -193,15 +167,4 @@ describe("eslint integration", function() {
193167
)
194168
})
195169
})
196-
197-
describe("output", function() {
198-
it("is not messed up", function() {
199-
this.timeout(5000)
200-
201-
executeConfig("output_mess/config.json")
202-
203-
expect(consoleMock.output).to.have.lengthOf(1)
204-
expect(consoleMock.output[0]).to.match(/^\{.*/)
205-
})
206-
})
207170
})

integration/ignore_warnings_test.js

-107
This file was deleted.

0 commit comments

Comments
 (0)