fix(app): init templates now place assert directives at the top of the generated .http files so it matches plugin expectations (#97)

Author: andrewmelchor

packages/app/src/cmd/init/basic.ts

@@ -175,12 +175,12 @@ describe('collection/users/list.http', () => {
 }
 
 export function generateCreatePostRequest(): string {
-  return `POST {{baseUrl}}/posts
+  return `# @assert status == 201
+
+POST {{baseUrl}}/posts
 Content-Type: application/json
 X-Request-ID: {{$uuid()}}
 
-# @assert status == 201
-
 {
   "title": "Hello from t-req",
   "body": "Created at {{$isodate()}}",
@@ -190,16 +190,16 @@ X-Request-ID: {{$uuid()}}
 }
 
 export function generateListUsersRequest(): string {
-  return `GET {{baseUrl}}/users
-Accept: application/json
+  return `# @assert status == 200
 
-# @assert status == 200
+GET {{baseUrl}}/users
+Accept: application/json
 `;
 }
 
 export function generateGetUserRequest(): string {
-  return `GET {{baseUrl}}/users/{{userId}}
+  return `# @assert status == 200
 
-# @assert status == 200
+GET {{baseUrl}}/users/{{userId}}
 `;
 }

packages/app/src/cmd/init/empty.ts

@@ -66,9 +66,9 @@ export function generateEmptyConfig(): string {
 }
 
 export function generateHelloRequest(): string {
-  return `GET {{baseUrl}}/users/1
+  return `# @assert status == 200
 
-# @assert status == 200
+GET {{baseUrl}}/users/1
 `;
 }
 

packages/app/test/cmd/init.test.ts

@@ -180,14 +180,23 @@ describe('generated file contents', () => {
     expect(createPost).toContain('{{baseUrl}}/posts');
     expect(createPost).toContain('"title"');
     expect(createPost).toContain('# @assert status == 201');
+    expect(createPost.indexOf('# @assert status == 201')).toBeLessThan(
+      createPost.indexOf('POST {{baseUrl}}/posts')
+    );
     expect(createPost).not.toContain('{{email}}');
     expect(createPost).not.toContain('{{password}}');
 
     const listUsers = generateListUsersRequest();
     expect(listUsers).toContain('# @assert status == 200');
+    expect(listUsers.indexOf('# @assert status == 200')).toBeLessThan(
+      listUsers.indexOf('GET {{baseUrl}}/users')
+    );
 
     const getUser = generateGetUserRequest();
     expect(getUser).toContain('# @assert status == 200');
+    expect(getUser.indexOf('# @assert status == 200')).toBeLessThan(
+      getUser.indexOf('GET {{baseUrl}}/users/{{userId}}')
+    );
   });
 
   test('should generate run script that imports from client', () => {
@@ -514,6 +523,9 @@ describe('empty template', () => {
     const request = generateHelloRequest();
     expect(request).toContain('GET {{baseUrl}}/users/1');
     expect(request).toContain('# @assert status == 200');
+    expect(request.indexOf('# @assert status == 200')).toBeLessThan(
+      request.indexOf('GET {{baseUrl}}/users/1')
+    );
   });
 
   test('all empty template variables should be defined in config', () => {