Skip to content

Commit 719a834

Browse files
[FSSDK-10616] util test case readability improvement
1 parent 3e28e68 commit 719a834

File tree

1 file changed

+23
-11
lines changed

1 file changed

+23
-11
lines changed

src/utils.spec.tsx

Lines changed: 23 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -25,28 +25,33 @@ describe('utils', () => {
2525

2626
it('returns true if users are equal', () => {
2727
const user2 = JSON.parse(JSON.stringify(user));
28+
const areUsersEqual = utils.areUsersEqual(user, user2);
2829

29-
expect(utils.areUsersEqual(user, user2)).toBe(true);
30+
expect(areUsersEqual).toBe(true);
3031
});
3132

3233
it('returns false if users are not equal', () => {
3334
const user2 = { id: '2', attributes: { name: 'user2' } };
35+
const areUsersEqual = utils.areUsersEqual(user, user2);
3436

35-
expect(utils.areUsersEqual(user, user2)).toBe(false);
37+
expect(areUsersEqual).toBe(false);
3638
});
3739

3840
it('returns false if key lengths are not equal', () => {
3941
const user2 = { id: '1', attributes: { name: 'user1', age: 30 } };
42+
const areUsersEqual = utils.areUsersEqual(user, user2);
4043

41-
expect(utils.areUsersEqual(user, user2)).toBe(false);
44+
expect(areUsersEqual).toBe(false);
4245
});
4346

4447
it('returns false if one of the key value pairs are not equal', () => {
4548
const user2 = { id: '1', attributes: { name: 'user2' } };
49+
const areUsersEqual = utils.areUsersEqual(user, user2);
4650

47-
expect(utils.areUsersEqual(user, user2)).toBe(false);
51+
expect(areUsersEqual).toBe(false);
4852
});
4953
});
54+
5055
describe('hoistStaticsAndForwardRefs', () => {
5156
class TestComponent extends React.Component<{ forwardedRef?: React.Ref<HTMLDivElement> }> {
5257
static testStaticMethod = () => 'static method result';
@@ -89,50 +94,57 @@ describe('utils', () => {
8994
it('should return true for equal attributes', () => {
9095
const attrs1 = { a: 1, b: 2 };
9196
const attrs2 = { a: 1, b: 2 };
97+
const areAttributesEqual = utils.areAttributesEqual(attrs1, attrs2);
9298

93-
expect(utils.areAttributesEqual(attrs1, attrs2)).toBe(true);
99+
expect(areAttributesEqual).toBe(true);
94100
});
95101

96102
it('should return false for different attribute keys', () => {
97103
const attrs1 = { a: 1, b: 2 };
98104
const attrs2 = { a: 1, c: 2 };
105+
const areAttributesEqual = utils.areAttributesEqual(attrs1, attrs2);
99106

100-
expect(utils.areAttributesEqual(attrs1, attrs2)).toBe(false);
107+
expect(areAttributesEqual).toBe(false);
101108
});
102109

103110
it('should return false for different attribute values', () => {
104111
const attrs1 = { a: 1, b: 2 };
105112
const attrs2 = { a: 1, b: 3 };
113+
const areAttributesEqual = utils.areAttributesEqual(attrs1, attrs2);
106114

107-
expect(utils.areAttributesEqual(attrs1, attrs2)).toBe(false);
115+
expect(areAttributesEqual).toBe(false);
108116
});
109117

110118
it('should return false if the number of attributes differs', () => {
111119
const attrs1 = { a: 1, b: 2 };
112120
const attrs2 = { a: 1 };
121+
const areAttributesEqual = utils.areAttributesEqual(attrs1, attrs2);
113122

114-
expect(utils.areAttributesEqual(attrs1, attrs2)).toBe(false);
123+
expect(areAttributesEqual).toBe(false);
115124
});
116125

117126
it('should handle undefined or null attributes as empty objects', () => {
118127
const attrs1 = null;
119128
const attrs2 = undefined;
129+
const areAttributesEqual = utils.areAttributesEqual(attrs1, attrs2);
120130

121-
expect(utils.areAttributesEqual(attrs1, attrs2)).toBe(true);
131+
expect(areAttributesEqual).toBe(true);
122132
});
123133

124134
it('should return false when one attribute is an object and another is not', () => {
125135
const attrs1 = { a: 1 };
126136
const attrs2 = 'not an object';
137+
const areAttributesEqual = utils.areAttributesEqual(attrs1, attrs2);
127138

128-
expect(utils.areAttributesEqual(attrs1, attrs2)).toBe(false);
139+
expect(areAttributesEqual).toBe(false);
129140
});
130141

131142
it('should handle different types of attribute values correctly', () => {
132143
const attrs1 = { a: '1', b: true };
133144
const attrs2 = { a: '1', b: true };
145+
const areAttributesEqual = utils.areAttributesEqual(attrs1, attrs2);
134146

135-
expect(utils.areAttributesEqual(attrs1, attrs2)).toBe(true);
147+
expect(areAttributesEqual).toBe(true);
136148
});
137149
});
138150

0 commit comments

Comments
 (0)