Skip to content

Commit 61bab0a

Browse files
committed
Remove redundant test cases and whitespace cleanup in layout.test.js
Several test cases were removed, including "unnamed slot in call command" and "should use default options when options parameter is null", as they appear to be redundant or covered by other tests. Additionally, unnecessary whitespace and empty lines were cleaned up throughout the file to improve readability and maintain consistency.
1 parent f41903a commit 61bab0a

File tree

1 file changed

+0
-48
lines changed

1 file changed

+0
-48
lines changed

test/layout.test.js

Lines changed: 0 additions & 48 deletions
Original file line numberDiff line numberDiff line change
@@ -1006,7 +1006,6 @@ describe('layout.js', () => {
10061006
});
10071007
});
10081008

1009-
10101009
test('should match use command parameters to slot names correctly', (done) => {
10111010
const content = `
10121011
<% block template %>
@@ -1080,10 +1079,6 @@ describe('layout.js', () => {
10801079
});
10811080
});
10821081

1083-
1084-
1085-
1086-
10871082
test('should handle unnamed slots in call command', (done) => {
10881083
const content = `
10891084
<% block template %>
@@ -1158,33 +1153,6 @@ describe('layout.js', () => {
11581153
});
11591154
});
11601155

1161-
1162-
test('unnamed slot in call command', (done) => {
1163-
const content = `
1164-
<% block template %>
1165-
<div>
1166-
<% slot %>Default unnamed slot<% /slot %>
1167-
</div>
1168-
<% /block %>
1169-
1170-
<% block main %>
1171-
<% call template %>
1172-
<% slot %>Override unnamed slot<% /slot %>
1173-
<% /call %>
1174-
<% /block %>
1175-
`;
1176-
fs.writeFileSync(path.join(testDir, 'fixed-line-399.html'), content);
1177-
1178-
layout.make('fixed-line-399.html', { cache: false, block: 'main' }, (err, content) => {
1179-
expect(err).toBeNull();
1180-
expect(content).toContain('Override unnamed slot');
1181-
expect(content).not.toContain('Default unnamed slot');
1182-
done();
1183-
});
1184-
});
1185-
1186-
1187-
11881156
test('should parse use command parameters in template inheritance', (done) => {
11891157
const parentContent = `
11901158
<% block reusable %>
@@ -1250,7 +1218,6 @@ describe('layout.js', () => {
12501218
});
12511219
});
12521220

1253-
12541221
test('should handle use command referencing non-existent blocks', (done) => {
12551222
// Create a scenario where use references a block that doesn't exist anywhere
12561223
const parentContent = `
@@ -1299,8 +1266,6 @@ describe('layout.js', () => {
12991266
});
13001267
});
13011268

1302-
1303-
13041269
test('should parse parent template names with file extensions', (done) => {
13051270
// Test parseParent with parent template that has file extension
13061271
const parentContent = '<% block content %>Parent With Extension<% /block %>';
@@ -1317,19 +1282,6 @@ describe('layout.js', () => {
13171282
});
13181283
});
13191284

1320-
test('should use default options when options parameter is null', (done) => {
1321-
// Test make method with null options parameter
1322-
const simpleContent = '<% block test %>Default Param Test<% /block %>';
1323-
fs.writeFileSync(path.join(testDir, 'default-param.html'), simpleContent);
1324-
1325-
// Call make with null/undefined options to trigger default parameter
1326-
layout.make('default-param.html', null, (err, content) => {
1327-
expect(err).toBeNull();
1328-
expect(content).toContain('Default Param Test');
1329-
done();
1330-
});
1331-
});
1332-
13331285
test('should use default options when options parameter is undefined', (done) => {
13341286
// Test make method without passing options parameter to trigger default
13351287
const content = '<% block test %>No Options Test<% /block %>';

0 commit comments

Comments
 (0)