Skip to content

Commit 8709c50

Browse files
author
Sal Niro
committedJun 14, 2017
slight cleanup
·
4.3.03.5.0
1 parent 414e7cb commit 8709c50

File tree

2 files changed

+161
-161
lines changed

2 files changed

+161
-161
lines changed
 

‎index.js

+2-2
Original file line numberDiff line numberDiff line change
@@ -28,8 +28,8 @@ module.exports = function gulpCleanCSS(options, callback) {
2828

2929
let contents = file.contents ? file.contents.toString() : '';
3030
let pass = {[file.path]: {styles: contents}};
31-
if (!options.rebaseTo && options.rebase!==false){
32-
options.rebaseTo = path.dirname(file.path);
31+
if (!options.rebaseTo && options.rebase !== false) {
32+
options.rebaseTo = path.dirname(file.path);
3333
}
3434

3535
new CleanCSS(options).minify(pass, function (errors, css) {

‎test/test.js

+159-159
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@ chai.use(require('chai-string'));
1717
describe('gulp-clean-css: init', function () {
1818

1919
it('should return the gulp-clean-css object: required export', function () {
20-
expect(cleanCSS).to.be.function;
20+
expect(cleanCSS).to.exist;
2121
});
2222
});
2323

@@ -27,54 +27,54 @@ describe('gulp-clean-css: base functionality', function () {
2727
var i = 0;
2828

2929
gulp.src(['test/fixtures/**/*.scss', '!test/fixtures/empty/**', '!test/fixtures/sourcemaps-load/**'])
30-
.pipe(gulpSass())
31-
.pipe(cleanCSS())
32-
.pipe(rename({
33-
suffix: '.generated',
34-
}))
35-
.pipe(gulp.dest('test/fixtures/'))
36-
.on('data', function (file) {
37-
i += 1;
38-
})
39-
.once('end', function () {
40-
i.should.equal(3);
41-
done();
42-
});
30+
.pipe(gulpSass())
31+
.pipe(cleanCSS())
32+
.pipe(rename({
33+
suffix: '.generated',
34+
}))
35+
.pipe(gulp.dest('test/fixtures/'))
36+
.on('data', function (file) {
37+
i += 1;
38+
})
39+
.once('end', function () {
40+
i.should.equal(3);
41+
done();
42+
});
4343
});
4444

4545
it('should allow the file through', function (done) {
4646
var i = 0;
4747

4848
gulp.src('test/fixtures/test.css')
49-
.pipe(cleanCSS())
50-
.on('data', function (file) {
51-
i += 1;
52-
})
53-
.once('end', function () {
54-
i.should.equal(1);
55-
done();
56-
});
49+
.pipe(cleanCSS())
50+
.on('data', function (file) {
51+
i += 1;
52+
})
53+
.once('end', function () {
54+
i.should.equal(1);
55+
done();
56+
});
5757
});
5858

5959
it('should allow the file through:empty file, pipe dest', function (done) {
6060
var i = 0;
6161

6262
gulp.src('test/fixtures/empty/**/*.scss')
63-
.pipe(gulpSass())
64-
.pipe(cleanCSS())
65-
.pipe(rename({
66-
suffix: '.generated',
67-
}))
68-
.pipe(gulp.dest(function (file) {
69-
return file.base + '/empty-parsed';
70-
}))
71-
.on('data', function (file) {
72-
i += 1;
73-
})
74-
.once('end', function () {
75-
i.should.equal(3);
76-
done();
77-
});
63+
.pipe(gulpSass())
64+
.pipe(cleanCSS())
65+
.pipe(rename({
66+
suffix: '.generated',
67+
}))
68+
.pipe(gulp.dest(function (file) {
69+
return file.base + '/empty-parsed';
70+
}))
71+
.on('data', function (file) {
72+
i += 1;
73+
})
74+
.once('end', function () {
75+
i.should.equal(3);
76+
done();
77+
});
7878
});
7979

8080
it('should produce the expected file', function (done) {
@@ -87,11 +87,11 @@ describe('gulp-clean-css: base functionality', function () {
8787
});
8888

8989
gulp.src('test/fixtures/test.css')
90-
.pipe(cleanCSS())
91-
.on('data', function (file) {
92-
file.contents.should.exist && expect(file.contents.toString()).to.equal(mockFile.contents.toString());
93-
done();
94-
});
90+
.pipe(cleanCSS())
91+
.on('data', function (file) {
92+
file.contents.should.exist && expect(file.contents.toString()).to.equal(mockFile.contents.toString());
93+
done();
94+
});
9595
});
9696

9797
it('should minify the css: empty file, no `file.contents`', function (done) {
@@ -106,111 +106,111 @@ describe('gulp-clean-css: base functionality', function () {
106106
});
107107

108108
vfsFake.src(mockFile)
109-
.pipe(cleanCSS())
110-
.on('data', function (file) {
111-
i += 1;
112-
})
113-
.once('end', function () {
114-
i.should.equal(1);
115-
done();
116-
});
109+
.pipe(cleanCSS())
110+
.on('data', function (file) {
111+
i += 1;
112+
})
113+
.once('end', function () {
114+
i.should.equal(1);
115+
done();
116+
});
117117
});
118118

119119
it('should invoke optional callback with details specified in options: debug', function (done) {
120120
gulp.src('test/fixtures/test.css')
121-
.pipe(cleanCSS({debug: true}, function (details) {
122-
details.stats.should.exist &&
123-
details.stats.originalSize.should.exist &&
124-
details.stats.minifiedSize.should.exist;
125-
}))
126-
.on('data', function (file) {
127-
done();
128-
});
121+
.pipe(cleanCSS({debug: true}, function (details) {
122+
details.stats.should.exist &&
123+
details.stats.originalSize.should.exist &&
124+
details.stats.minifiedSize.should.exist;
125+
}))
126+
.on('data', function (file) {
127+
done();
128+
});
129129
});
130130

131131
it('should invoke optional callback with out options object supplied: return object hash', function (done) {
132132
gulp.src('test/fixtures/test.css')
133-
.pipe(cleanCSS(function (details) {
134-
details.stats.should.exist &&
135-
expect(details).to.have.ownProperty('stats') &&
136-
expect(details).to.have.ownProperty('errors') &&
137-
expect(details).to.have.ownProperty('warnings') &&
138-
expect(details).to.not.have.ownProperty('sourceMap');
139-
}))
140-
.on('data', function (file) {
141-
done();
142-
});
133+
.pipe(cleanCSS(function (details) {
134+
details.stats.should.exist &&
135+
expect(details).to.have.ownProperty('stats') &&
136+
expect(details).to.have.ownProperty('errors') &&
137+
expect(details).to.have.ownProperty('warnings') &&
138+
expect(details).to.not.have.ownProperty('sourceMap');
139+
}))
140+
.on('data', function (file) {
141+
done();
142+
});
143143
});
144144

145145
it('should invoke optional callback without options object supplied: return object hash with sourceMap: true; return correct hash', function (done) {
146146
gulp.src('test/fixtures/test.css')
147-
.pipe(cleanCSS({sourceMap: true}, function (details) {
148-
details.stats.should.exist &&
149-
expect(details).have.ownProperty('sourceMap');
150-
}))
151-
.on('data', function (file) {
152-
done();
153-
});
147+
.pipe(cleanCSS({sourceMap: true}, function (details) {
148+
details.stats.should.exist &&
149+
expect(details).have.ownProperty('sourceMap');
150+
}))
151+
.on('data', function (file) {
152+
done();
153+
});
154154
});
155155

156156
it('should invoke optional callback with file details returned', function (done) {
157157

158158
var expected = 'test.css'
159159

160160
gulp.src('test/fixtures/test.css')
161-
.pipe(cleanCSS(function (details) {
162-
details.name.should.equal(expected)
163-
}))
164-
.on('data', function (file) {
165-
done();
166-
});
161+
.pipe(cleanCSS(function (details) {
162+
details.name.should.equal(expected)
163+
}))
164+
.on('data', function (file) {
165+
done();
166+
});
167167
});
168168

169169
it('should write sourcemaps', function (done) {
170170

171171
var i = 0;
172172

173173
gulp.src(['test/fixtures/sourcemaps/**/*.css', '!test/fixtures/sourcemaps/**/*.generated.css'])
174-
.pipe(sourcemaps.init())
175-
.pipe(concat('sourcemapped.css'))
176-
.pipe(cleanCSS())
177-
.pipe(rename({
178-
suffix: '.generated',
179-
}))
180-
.on('data', function (file) {
181-
i += 1;
182-
})
183-
.pipe(sourcemaps.write())
184-
.pipe(gulp.dest(function (file) {
185-
return file.base;
186-
}))
187-
.once('end', function () {
188-
i.should.equal(1);
189-
done();
190-
});
174+
.pipe(sourcemaps.init())
175+
.pipe(concat('sourcemapped.css'))
176+
.pipe(cleanCSS())
177+
.pipe(rename({
178+
suffix: '.generated',
179+
}))
180+
.on('data', function (file) {
181+
i += 1;
182+
})
183+
.pipe(sourcemaps.write())
184+
.pipe(gulp.dest(function (file) {
185+
return file.base;
186+
}))
187+
.once('end', function () {
188+
i.should.equal(1);
189+
done();
190+
});
191191
});
192192

193193
it('should write sourcemaps, worrectly map output', function (done) {
194194

195195
var i = 0;
196196

197197
gulp.src('test/fixtures/sourcemaps-load/scss/test-sass.scss')
198-
.pipe(sourcemaps.init())
199-
.pipe(gulpSass())
200-
.pipe(sourcemaps.init({loadMaps: true}))
201-
.pipe(cleanCSS({sourceMapInlineSources: true}))
202-
.on('data', function (file) {
203-
i += 1;
204-
})
205-
.pipe(rename({
206-
suffix: '.min'
207-
}))
208-
.pipe(sourcemaps.write())
209-
.pipe(gulp.dest('test/fixtures/sourcemaps-load/min'))
210-
.once('end', function () {
211-
i.should.equal(1); // todo inspect mapping here
212-
done();
213-
});
198+
.pipe(sourcemaps.init())
199+
.pipe(gulpSass())
200+
.pipe(sourcemaps.init({loadMaps: true}))
201+
.pipe(cleanCSS({sourceMapInlineSources: true}))
202+
.on('data', function (file) {
203+
i += 1;
204+
})
205+
.pipe(rename({
206+
suffix: '.min'
207+
}))
208+
.pipe(sourcemaps.write())
209+
.pipe(gulp.dest('test/fixtures/sourcemaps-load/min'))
210+
.once('end', function () {
211+
i.should.equal(1); // todo inspect mapping here
212+
done();
213+
});
214214
});
215215

216216
it('should return a warning for improper syntax', function (done) {
@@ -223,28 +223,28 @@ describe('gulp-clean-css: base functionality', function () {
223223
});
224224

225225
vfsFake.src(css)
226-
.pipe(cleanCSS({debug: true}, function (details) {
227-
expect(details.warnings).to.exist &&
228-
expect(details.warnings.length).to.equal(1) &&
229-
expect(details.warnings[0]).to.equal('Missing \'}\' at fixtures/test.css:1:14.');
230-
}))
231-
.on('data', function (file) {
232-
i += 1;
233-
})
234-
.once('end', function () {
235-
i.should.equal(1);
236-
done();
237-
});
226+
.pipe(cleanCSS({debug: true}, function (details) {
227+
expect(details.warnings).to.exist &&
228+
expect(details.warnings.length).to.equal(1) &&
229+
expect(details.warnings[0]).to.equal('Missing \'}\' at fixtures/test.css:1:14.');
230+
}))
231+
.on('data', function (file) {
232+
i += 1;
233+
})
234+
.once('end', function () {
235+
i.should.equal(1);
236+
done();
237+
});
238238
});
239239

240240
it('should invoke a plugin error: streaming not supported', function (done) {
241241

242242
gulp.src('test/fixtures/test.css', {buffer: false})
243-
.pipe(cleanCSS()
244-
.on('error', function (err) {
245-
expect(err.message).to.equal('Streaming not supported!')
246-
done();
247-
}));
243+
.pipe(cleanCSS()
244+
.on('error', function (err) {
245+
expect(err.message).to.equal('Streaming not supported!')
246+
done();
247+
}));
248248
});
249249

250250
it('should return a clean-css error', function (done) {
@@ -255,12 +255,12 @@ describe('gulp-clean-css: base functionality', function () {
255255
});
256256

257257
vfsFake.src(css)
258-
.pipe(cleanCSS())
259-
.on('error', function (err) {
260-
expect(err).to.exist;
261-
expect(err).to.equal('Ignoring local @import of "/some/fake/file" as resource is missing.');
262-
done();
263-
});
258+
.pipe(cleanCSS())
259+
.on('error', function (err) {
260+
expect(err).to.exist;
261+
expect(err).to.equal('Ignoring local @import of "/some/fake/file" as resource is missing.');
262+
done();
263+
});
264264
});
265265
});
266266

@@ -269,50 +269,50 @@ describe('gulp-clean-css: rebase', function () {
269269
it('should not rebase files by default - do not resolve relative files', function (done) {
270270

271271
gulp.src(['test/fixtures/rebasing/subdir/insub.css'])
272-
.pipe(cleanCSS({rebase: false}))
273-
.on('data', function (file) {
272+
.pipe(cleanCSS({rebase: false}))
273+
.on('data', function (file) {
274274

275-
let expected = `
275+
let expected = `
276276
p.insub_same{background:url(insub.png)}
277277
p.insub_child{background:url(child/child.png)}
278278
p.insub_parent{background:url(../parent.png)}
279279
p.insub_other{background:url(../othersub/inother.png)}
280280
p.insub_absolute{background:url(/inroot.png)}`;
281281

282-
let actual = file.contents.toString();
282+
let actual = file.contents.toString();
283283

284-
expect(actual).to.equalIgnoreSpaces(expected)
285-
})
286-
.once('end', done);
284+
expect(actual).to.equalIgnoreSpaces(expected)
285+
})
286+
.once('end', done);
287287
});
288288

289289
it('should by rebase files with target specified', function (done) {
290290

291291
gulp.src(['test/fixtures/rebasing/subdir/insub.css'])
292-
.pipe(cleanCSS({rebaseTo: 'test'}))
293-
.on('data', function (file) {
292+
.pipe(cleanCSS({rebaseTo: 'test'}))
293+
.on('data', function (file) {
294294

295-
let expected = `
295+
let expected = `
296296
p.insub_same{background:url(fixtures/rebasing/subdir/insub.png)}
297297
p.insub_child{background:url(fixtures/rebasing/subdir/child/child.png)}
298298
p.insub_parent{background:url(fixtures/rebasing/parent.png)}
299299
p.insub_other{background:url(fixtures/rebasing/othersub/inother.png)}
300300
p.insub_absolute{background:url(/inroot.png)}`;
301301

302-
let actual = file.contents.toString();
302+
let actual = file.contents.toString();
303303

304-
expect(actual).to.equalIgnoreSpaces(expected);
305-
})
306-
.once('end', done);
304+
expect(actual).to.equalIgnoreSpaces(expected);
305+
})
306+
.once('end', done);
307307
});
308-
308+
309309
it('should rebase to current relative file location - relative imports are resolved like in the browser', function (done) {
310310

311311
gulp.src(['test/fixtures/rebasing/subdir/import.css'])
312-
.pipe(cleanCSS())
313-
.on('data', function (file) {
312+
.pipe(cleanCSS())
313+
.on('data', function (file) {
314314

315-
let expected = `
315+
let expected = `
316316
p.imported_nested{background:url(../otherdir/nestedsub/nested.png)}
317317
p.imported_same{background:url(../otherdir/imported.png)}
318318
p.imported_parent{background:url(../parent.png)}
@@ -325,10 +325,10 @@ describe('gulp-clean-css: rebase', function () {
325325
p.insub_absolute{background:url(/inroot.png)}
326326
p.import{background:url(import.png)}`;
327327

328-
let actual = file.contents.toString();
328+
let actual = file.contents.toString();
329329

330-
expect(actual).to.equalIgnoreSpaces(expected)
331-
})
332-
.once('end', done);
330+
expect(actual).to.equalIgnoreSpaces(expected)
331+
})
332+
.once('end', done);
333333
});
334334
});

0 commit comments

Comments
 (0)
Please sign in to comment.