WRO-266: Unskip one of Skinnable unit test cases (#3056)

* updated skipped test for Skinnable

* lint warning fixes

* lint warning fixes

* Update unit test

Enact-DCO-1.0-Signed-off-by: Seungcheon Baek (sc.baek@lge.com)

* Fix lint warnings

Enact-DCO-1.0-Signed-off-by: Seungcheon Baek (sc.baek@lge.com)

Co-authored-by: Daniel Stoian <daniel.stoian@lgepartner.com>

packages/ui/Skinnable/tests/Skinnable-specs.js

@@ -376,7 +376,7 @@ describe('Skinnable Specs', () => {
 		expect(component).toHaveClass(expected);
 	});
 
-	test.skip('should not force re-render of child if child unaffected', () => {
+	test('should not force re-render of child if child unaffected', () => {
 		const config = {
 			defaultSkin: 'dark',
 			defaultVariants: 'normal',
@@ -400,19 +400,33 @@ describe('Skinnable Specs', () => {
 		const SkinnableParent = Skinnable(config, Component);
 		const SkinnableChild = Skinnable(config, ChildComponent);
 
-		// eslint-disable-next-line
-		const subject = mount(
+		const childrenProp = (<SkinnableChild />);
+
+		const {rerender} = render(
 			<SkinnableParent>
-				<SkinnableChild />
+				{childrenProp}
+			</SkinnableParent>
+		);
+
+		rerender(
+			<SkinnableParent className="foo">
+				{childrenProp}
 			</SkinnableParent>
 		);
 
-		// Sending props to force parent to re-render
-		subject.setProps({className: 'foo'});
+		const expected1 = 1;
+		const actual1 = wasRendered.mock.calls.length;
+
+		rerender(
+			<SkinnableParent className="foo" skin="light">
+				{childrenProp}
+			</SkinnableParent>
+		);
 
-		const expected = 1;
-		const actual = wasRendered.mock.calls.length;
+		const expected2 = 2;
+		const actual2 = wasRendered.mock.calls.length;
 
-		expect(actual).toEqual(expected);
+		expect(actual1).toEqual(expected1);
+		expect(actual2).toEqual(expected2);
 	});
 });