Skip to content

Commit 39d4a14

Browse files
laveesinghlaveesingh
committed
prettified entire codebase
1 parent 208d549 commit 39d4a14

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

67 files changed

+3391
-2352
lines changed

‎__tests__/SimpleSlider.test.js

+25-26
Original file line numberDiff line numberDiff line change
@@ -1,35 +1,34 @@
11
// includes tests of
22
// SimpleSlider, MultipleItems
3-
import {mount} from 'enzyme'
4-
import {
5-
testSlider
6-
} from './testUtils'
3+
import { mount } from "enzyme";
4+
import { testSlider } from "./testUtils";
75

8-
describe('SimpleSlider with combinations of possibilities', function(){
6+
describe("SimpleSlider with combinations of possibilities", function() {
97
// try around several possibilities
10-
let _noOfSlides = [2, 5, 12]
11-
let _slidesToShow = [2, 5, 10]
12-
let _slidesToScroll = [1, 2, 3, 10]
13-
if(true){ // for switching real quick to lesser/easier tests for simplicity
14-
_noOfSlides = [5]
15-
_slidesToShow = [2]
16-
_slidesToScroll = [1, 2]
8+
let _noOfSlides = [2, 5, 12];
9+
let _slidesToShow = [2, 5, 10];
10+
let _slidesToScroll = [1, 2, 3, 10];
11+
if (true) {
12+
// for switching real quick to lesser/easier tests for simplicity
13+
_noOfSlides = [5];
14+
_slidesToShow = [2];
15+
_slidesToScroll = [1, 2];
1716
}
1817

19-
for(let noOfSlides of _noOfSlides){
20-
for(let slidesToShow of _slidesToShow){
21-
for(let slidesToScroll of _slidesToScroll){
18+
for (let noOfSlides of _noOfSlides) {
19+
for (let slidesToShow of _slidesToShow) {
20+
for (let slidesToScroll of _slidesToScroll) {
2221
// following restrictions may not be 100% correct, and there may be more restrictions
23-
if(slidesToShow > noOfSlides || slidesToScroll > slidesToShow) {
24-
continue
22+
if (slidesToShow > noOfSlides || slidesToScroll > slidesToShow) {
23+
continue;
2524
}
26-
if(noOfSlides === slidesToShow){
25+
if (noOfSlides === slidesToShow) {
2726
// temporary, jquery slick disables arrows in this case, so the tests fail
28-
continue
27+
continue;
2928
}
30-
if(slidesToShow === slidesToScroll){
29+
if (slidesToShow === slidesToScroll) {
3130
// temporary, active-class is not being assigned properly, so tests fail
32-
continue
31+
continue;
3332
}
3433
const settings1 = {
3534
infinite: true,
@@ -38,11 +37,11 @@ describe('SimpleSlider with combinations of possibilities', function(){
3837
slidesToShow,
3938
slidesToScroll,
4039
useCSS: false
41-
}
42-
test(`Test with settings => noOfSlides: ${noOfSlides}, slidesToShow: ${slidesToShow}, slidesToScroll: ${slidesToScroll}`, function(){
43-
testSlider(settings1)
44-
})
40+
};
41+
test(`Test with settings => noOfSlides: ${noOfSlides}, slidesToShow: ${slidesToShow}, slidesToScroll: ${slidesToScroll}`, function() {
42+
testSlider(settings1);
43+
});
4544
}
4645
}
4746
}
48-
})
47+
});

‎__tests__/afterChange.test.js

+15-15
Original file line numberDiff line numberDiff line change
@@ -1,20 +1,20 @@
1-
import React from 'react';
2-
import {mount} from 'enzyme';
3-
import Slider from '../src/index';
1+
import React from "react";
2+
import { mount } from "enzyme";
3+
import Slider from "../src/index";
44

55
class SliderWithBeforeChange extends React.Component {
66
constructor(props) {
7-
super(props)
7+
super(props);
88
this.state = {
9-
currentSlide: null,
10-
}
11-
this.afterChange = this.afterChange.bind(this)
9+
currentSlide: null
10+
};
11+
this.afterChange = this.afterChange.bind(this);
1212
}
1313
afterChange(currentSlide) {
14-
console.log(currentSlide, 'afterChange')
14+
console.log(currentSlide, "afterChange");
1515
this.setState({
16-
currentSlide,
17-
})
16+
currentSlide
17+
});
1818
}
1919
render() {
2020
return (
@@ -24,15 +24,15 @@ class SliderWithBeforeChange extends React.Component {
2424
<div>slide3</div>
2525
<div>slide4</div>
2626
</Slider>
27-
)
27+
);
2828
}
2929
}
3030

31-
describe('After change Slider', function() {
32-
it('should render', function() {
31+
describe("After change Slider", function() {
32+
it("should render", function() {
3333
const wrapper = mount(<SliderWithBeforeChange />);
34-
expect(wrapper.state()).toEqual({currentSlide: null})
35-
wrapper.find('.slick-next').simulate('click')
34+
expect(wrapper.state()).toEqual({ currentSlide: null });
35+
wrapper.find(".slick-next").simulate("click");
3636

3737
//TBD . fix this test
3838

‎__tests__/arrows.js

+14-13
Original file line numberDiff line numberDiff line change
@@ -2,30 +2,31 @@
22
* Arrow component tests
33
*/
44

5-
sinon.stub(console, 'error');
5+
sinon.stub(console, "error");
66

7-
import {render, shallow} from 'enzyme';
8-
import React from 'react';
9-
import sinon from 'sinon';
7+
import { render, shallow } from "enzyme";
8+
import React from "react";
9+
import sinon from "sinon";
1010

11-
import { NextArrow, PrevArrow } from '../src/arrows';
11+
import { NextArrow, PrevArrow } from "../src/arrows";
1212

1313
function CustomArrow(props) {
1414
return (
1515
<span
1616
className="sample"
1717
data-currentSlide={props.currentSlide}
18-
data-slideCount={props.slideCount} />
18+
data-slideCount={props.slideCount}
19+
/>
1920
);
2021
}
2122

22-
describe('Previous arrows', () => {
23-
it('should render arrow', () => {
23+
describe("Previous arrows", () => {
24+
it("should render arrow", () => {
2425
const wrapper = shallow(<PrevArrow />);
25-
expect(wrapper.find('button')).toHaveLength(1);
26+
expect(wrapper.find("button")).toHaveLength(1);
2627
});
2728

28-
it('should not result in errors', () => {
29+
it("should not result in errors", () => {
2930
shallow(<PrevArrow />);
3031

3132
expect(console.error.called).toBe(false);
@@ -43,10 +44,10 @@ describe('Previous arrows', () => {
4344
// });
4445
});
4546

46-
describe('Next arrows', () => {
47-
it('should render arrow', () => {
47+
describe("Next arrows", () => {
48+
it("should render arrow", () => {
4849
const wrapper = shallow(<NextArrow />);
49-
expect(wrapper.find('button')).toHaveLength(1);
50+
expect(wrapper.find("button")).toHaveLength(1);
5051
});
5152

5253
// it('should not result in errors', () => {

‎__tests__/beforeChange.test.js

+34-19
Original file line numberDiff line numberDiff line change
@@ -1,21 +1,21 @@
1-
import React from 'react';
2-
import {mount} from 'enzyme';
3-
import Slider from '../src/index';
1+
import React from "react";
2+
import { mount } from "enzyme";
3+
import Slider from "../src/index";
44

55
class SliderWithBeforeChange extends React.Component {
66
constructor(props) {
7-
super(props)
7+
super(props);
88
this.state = {
99
currentSlide: null,
1010
nextSlide: null
11-
}
12-
this.beforeChange = this.beforeChange.bind(this)
11+
};
12+
this.beforeChange = this.beforeChange.bind(this);
1313
}
1414
beforeChange(currentSlide, nextSlide) {
1515
this.setState({
1616
currentSlide,
1717
nextSlide
18-
})
18+
});
1919
}
2020
render() {
2121
return (
@@ -25,21 +25,36 @@ class SliderWithBeforeChange extends React.Component {
2525
<div>slide3</div>
2626
<div>slide4</div>
2727
</Slider>
28-
)
28+
);
2929
}
3030
}
3131

32-
describe.skip('Slider', function() {
33-
it('should render', function() {
32+
describe.skip("Slider", function() {
33+
it("should render", function() {
3434
const wrapper = mount(<SliderWithBeforeChange />);
35-
wrapper.find('.slick-next').simulate('click')
36-
expect(wrapper.find('.slick-slide.slick-active').first().text()).toEqual('slide2');
37-
expect(wrapper.state()).toEqual({currentSlide: 0, nextSlide: 1})
38-
wrapper.find('.slick-next').simulate('click')
39-
expect(wrapper.find('.slick-slide.slick-active').first().text()).toEqual('slide3');
40-
expect(wrapper.state()).toEqual({currentSlide: 1, nextSlide: 2})
41-
wrapper.find('.slick-prev').simulate('click')
42-
expect(wrapper.find('.slick-slide.slick-active').first().text()).toEqual('slide2');
43-
expect(wrapper.state()).toEqual({currentSlide: 2, nextSlide: 1})
35+
wrapper.find(".slick-next").simulate("click");
36+
expect(
37+
wrapper
38+
.find(".slick-slide.slick-active")
39+
.first()
40+
.text()
41+
).toEqual("slide2");
42+
expect(wrapper.state()).toEqual({ currentSlide: 0, nextSlide: 1 });
43+
wrapper.find(".slick-next").simulate("click");
44+
expect(
45+
wrapper
46+
.find(".slick-slide.slick-active")
47+
.first()
48+
.text()
49+
).toEqual("slide3");
50+
expect(wrapper.state()).toEqual({ currentSlide: 1, nextSlide: 2 });
51+
wrapper.find(".slick-prev").simulate("click");
52+
expect(
53+
wrapper
54+
.find(".slick-slide.slick-active")
55+
.first()
56+
.text()
57+
).toEqual("slide2");
58+
expect(wrapper.state()).toEqual({ currentSlide: 2, nextSlide: 1 });
4459
});
4560
});

‎__tests__/index.js

+18-10
Original file line numberDiff line numberDiff line change
@@ -1,14 +1,22 @@
1-
'use strict';
1+
"use strict";
22

3-
import React from 'react';
4-
import {shallow, mount} from 'enzyme';
5-
import Slider from '../src/index';
3+
import React from "react";
4+
import { shallow, mount } from "enzyme";
5+
import Slider from "../src/index";
66

7-
describe('Slider', function() {
8-
it('should render', function() {
9-
const wrapper = shallow(<Slider><div>slide1</div></Slider>);
10-
expect(wrapper.contains(
11-
<div tabIndex={-1} style={{ width: '100%', display: 'inline-block' }}>slide1</div>
12-
)).toBe(true);
7+
describe("Slider", function() {
8+
it("should render", function() {
9+
const wrapper = shallow(
10+
<Slider>
11+
<div>slide1</div>
12+
</Slider>
13+
);
14+
expect(
15+
wrapper.contains(
16+
<div tabIndex={-1} style={{ width: "100%", display: "inline-block" }}>
17+
slide1
18+
</div>
19+
)
20+
).toBe(true);
1321
});
1422
});

0 commit comments

Comments
 (0)