test: ✅ rename tests

Author: yjaaidi

apps/101-meal-planner-solution/src/app/meal-planner/meal-planner.spec.ts

@@ -3,7 +3,7 @@ import { recipeMother } from '../testing/recipe.mother';
 import { MealPlanner } from './meal-planner';
 
 describe(MealPlanner.name, () => {
-  it('should add recipe', () => {
+  it('adds recipes', () => {
     const { mealPlanner, burger, salad } = createMealPlanner();
 
     mealPlanner.addRecipe(burger);
@@ -15,19 +15,19 @@ describe(MealPlanner.name, () => {
     ]);
   });
 
-  it('should not allow recipe duplicates', () => {
+  it('does not allow recipe duplicates', () => {
     const { mealPlanner, burgerDuplicate } = createMealPlannerWithBurger();
 
     expect(mealPlanner.canAddRecipe(burgerDuplicate)).toBe(false);
   });
 
-  it('should allow new recipes', () => {
+  it('allows new recipes', () => {
     const { mealPlanner, salad } = createMealPlannerWithBurger();
 
     expect(mealPlanner.canAddRecipe(salad)).toBe(true);
   });
 
-  it('should throw error if recipe is already present', () => {
+  it('throws error if recipe is already present', () => {
     const { mealPlanner, burgerDuplicate } = createMealPlannerWithBurger();
 
     expect(() => mealPlanner.addRecipe(burgerDuplicate)).toThrow(

apps/102-meal-planner-reactive-solution/src/app/meal-planner/meal-planner.spec.ts

@@ -4,7 +4,7 @@ import { recipeMother } from '../testing/recipe.mother';
 import { MealPlanner } from './meal-planner';
 
 describe(MealPlanner.name, () => {
-  it('should add recipe', () => {
+  it('adds recipes', () => {
     const { mealPlanner, burger, salad } = createMealPlanner();
 
     mealPlanner.addRecipe(burger);
@@ -16,19 +16,19 @@ describe(MealPlanner.name, () => {
     ]);
   });
 
-  it('should not allow recipe duplicates', () => {
+  it('does not allow recipe duplicates', () => {
     const { mealPlanner, burgerDuplicate } = createMealPlannerWithBurger();
 
     expect(mealPlanner.canAddRecipe(burgerDuplicate)).toBe(false);
   });
 
-  it('should allow new recipes', () => {
+  it('allows new recipes', () => {
     const { mealPlanner, salad } = createMealPlannerWithBurger();
 
     expect(mealPlanner.canAddRecipe(salad)).toBe(true);
   });
 
-  it('should throw error if recipe is already present', () => {
+  it('throws error if recipe is already present', () => {
     const { mealPlanner, burgerDuplicate } = createMealPlannerWithBurger();
 
     expect(() => mealPlanner.addRecipe(burgerDuplicate)).toThrow(
@@ -37,7 +37,7 @@ describe(MealPlanner.name, () => {
   });
 
   describe('recipes$', () => {
-    it('should emit empty array when no recipes', async () => {
+    it('emits empty array when no recipes', async () => {
       const { mealPlanner } = createMealPlanner();
 
       using observer = observe(mealPlanner.recipes$);
@@ -46,7 +46,7 @@ describe(MealPlanner.name, () => {
       expect(observer.next).toHaveBeenCalledWith([]);
     });
 
-    it('should emit recipes when added', () => {
+    it('emits recipes when added', () => {
       const { mealPlanner, burger, salad } = createMealPlanner();
 
       using observer = observe(mealPlanner.recipes$);
@@ -68,7 +68,7 @@ describe(MealPlanner.name, () => {
   });
 
   describe('watchCanAddRecipe()', () => {
-    it('should instantly emit if recipe can be added', () => {
+    it('emits instantly if recipe can be added', () => {
       const { mealPlanner, burger } = createMealPlanner();
 
       using observer = observe(mealPlanner.watchCanAddRecipe(burger));

apps/102-meal-planner-reactive-starter/src/app/meal-planner/meal-planner.spec.ts

@@ -3,7 +3,7 @@ import { recipeMother } from '../testing/recipe.mother';
 import { MealPlanner } from './meal-planner';
 
 describe(MealPlanner.name, () => {
-  it('should add recipe', () => {
+  it('adds recipes', () => {
     const { mealPlanner, burger, salad } = createMealPlanner();
 
     mealPlanner.addRecipe(burger);
@@ -15,19 +15,19 @@ describe(MealPlanner.name, () => {
     ]);
   });
 
-  it('should not allow recipe duplicates', () => {
+  it('does not allow recipe duplicates', () => {
     const { mealPlanner, burgerDuplicate } = createMealPlannerWithBurger();
 
     expect(mealPlanner.canAddRecipe(burgerDuplicate)).toBe(false);
   });
 
-  it('should allow new recipes', () => {
+  it('allows new recipes', () => {
     const { mealPlanner, salad } = createMealPlannerWithBurger();
 
     expect(mealPlanner.canAddRecipe(salad)).toBe(true);
   });
 
-  it('should throw error if recipe is already present', () => {
+  it('throws error if recipe is already present', () => {
     const { mealPlanner, burgerDuplicate } = createMealPlannerWithBurger();
 
     expect(() => mealPlanner.addRecipe(burgerDuplicate)).toThrow(

apps/201-meal-planner-indirect-output-solution/src/app/meal-planner/meal-planner.spec.ts

@@ -9,7 +9,7 @@ import {
 } from './meal-repository.fake';
 
 describe(MealPlanner.name, () => {
-  it('should add recipe', async () => {
+  it('adds recipes', async () => {
     const { mealPlanner, burger, salad } = createMealPlanner();
 
     mealPlanner.addRecipe(burger);
@@ -21,15 +21,15 @@ describe(MealPlanner.name, () => {
     ]);
   });
 
-  it('should throw error if recipe is already present', () => {
+  it('throws error if recipe is already present', () => {
     const { mealPlanner, burgerDuplicate } = createMealPlannerWithBurger();
 
     expect(() => mealPlanner.addRecipe(burgerDuplicate)).toThrow(
       `Can't add recipe.`,
     );
   });
 
-  it('should add recipe to meal repository', () => {
+  it('adds recipe to meal repository', () => {
     const { mealPlanner, mealRepoFake, burger } = createMealPlanner();
 
     mealPlanner.addRecipe(burger);
@@ -40,7 +40,7 @@ describe(MealPlanner.name, () => {
   });
 
   describe('recipes$', () => {
-    it('should emit empty array when no recipes', async () => {
+    it('emits empty array when no recipes', async () => {
       const { mealPlanner } = createMealPlanner();
 
       using observer = observe(mealPlanner.recipes$);
@@ -49,7 +49,7 @@ describe(MealPlanner.name, () => {
       expect(observer.next).toHaveBeenCalledWith([]);
     });
 
-    it('should emit recipes when added', () => {
+    it('emits recipes when added', () => {
       const { mealPlanner, burger, salad } = createMealPlanner();
 
       using observer = observe(mealPlanner.recipes$);
@@ -71,7 +71,7 @@ describe(MealPlanner.name, () => {
   });
 
   describe('watchCanAddRecipe()', () => {
-    it('should instantly emit if recipe can be added', () => {
+    it('emits instantly if recipe can be added', () => {
       const { mealPlanner, burger } = createMealPlanner();
 
       using observer = observe(mealPlanner.watchCanAddRecipe(burger));

apps/201-meal-planner-indirect-output-starter/src/app/meal-planner/meal-planner.spec.ts

@@ -5,7 +5,7 @@ import { recipeMother } from '../testing/recipe.mother';
 import { MealPlanner } from './meal-planner';
 
 describe(MealPlanner.name, () => {
-  it('should add recipe', async () => {
+  it('adds recipes', async () => {
     const { mealPlanner, burger, salad } = createMealPlanner();
 
     mealPlanner.addRecipe(burger);
@@ -17,7 +17,7 @@ describe(MealPlanner.name, () => {
     ]);
   });
 
-  it('should throw error if recipe is already present', () => {
+  it('throws error if recipe is already present', () => {
     const { mealPlanner, burgerDuplicate } = createMealPlannerWithBurger();
 
     expect(() => mealPlanner.addRecipe(burgerDuplicate)).toThrow(
@@ -28,7 +28,7 @@ describe(MealPlanner.name, () => {
   it.todo('🚧 should add recipe to meal repository');
 
   describe('recipes$', () => {
-    it('should emit empty array when no recipes', async () => {
+    it('emits empty array when no recipes', async () => {
       const { mealPlanner } = createMealPlanner();
 
       using observer = observe(mealPlanner.recipes$);
@@ -37,7 +37,7 @@ describe(MealPlanner.name, () => {
       expect(observer.next).toHaveBeenCalledWith([]);
     });
 
-    it('should emit recipes when added', () => {
+    it('emits recipes when added', () => {
       const { mealPlanner, burger, salad } = createMealPlanner();
 
       using observer = observe(mealPlanner.recipes$);
@@ -59,7 +59,7 @@ describe(MealPlanner.name, () => {
   });
 
   describe('watchCanAddRecipe()', () => {
-    it('should instantly emit if recipe can be added', () => {
+    it('emits instantly if recipe can be added', () => {
       const { mealPlanner, burger } = createMealPlanner();
 
       using observer = observe(mealPlanner.watchCanAddRecipe(burger));

apps/202-meal-planner-indirect-input-solution/src/app/meal-planner/meal-planner.spec.ts

@@ -9,7 +9,7 @@ import {
 } from './meal-repository.fake';
 
 describe(MealPlanner.name, () => {
-  it('should add recipe', async () => {
+  it('adds recipes', async () => {
     const { mealPlanner, burger, salad } = createMealPlanner();
 
     mealPlanner.addRecipe(burger);
@@ -21,15 +21,15 @@ describe(MealPlanner.name, () => {
     ]);
   });
 
-  it('should throw error if recipe is already present', () => {
+  it('throws error if recipe is already present', () => {
     const { mealPlanner, burgerDuplicate } = createMealPlannerWithBurger();
 
     expect(() => mealPlanner.addRecipe(burgerDuplicate)).toThrow(
       `Can't add recipe.`,
     );
   });
 
-  it('should add recipe to meal repository', () => {
+  it('adds recipe to meal repository', () => {
     const { mealPlanner, mealRepoFake, burger } = createMealPlanner();
 
     mealPlanner.addRecipe(burger);
@@ -39,7 +39,7 @@ describe(MealPlanner.name, () => {
     ]);
   });
 
-  it('should fetch recipes from meal repository', async () => {
+  it('fetches recipes from meal repository', async () => {
     const { getMealPlanner, mealRepoFake, burger } = setUpMealPlanner();
 
     mealRepoFake.addMeal(burger);
@@ -51,7 +51,7 @@ describe(MealPlanner.name, () => {
   });
 
   describe('recipes$', () => {
-    it('should emit empty array when no recipes', async () => {
+    it('emits empty array when no recipes', async () => {
       const { mealPlanner } = createMealPlanner();
 
       using observer = observe(mealPlanner.recipes$);
@@ -60,7 +60,7 @@ describe(MealPlanner.name, () => {
       expect(observer.next).toHaveBeenCalledWith([]);
     });
 
-    it('should emit recipes when added', () => {
+    it('emits recipes when added', () => {
       const { mealPlanner, burger, salad } = createMealPlanner();
 
       using observer = observe(mealPlanner.recipes$);
@@ -82,7 +82,7 @@ describe(MealPlanner.name, () => {
   });
 
   describe('watchCanAddRecipe()', () => {
-    it('should instantly emit if recipe can be added', () => {
+    it('emits instantly if recipe can be added', () => {
       const { mealPlanner, burger } = createMealPlanner();
 
       using observer = observe(mealPlanner.watchCanAddRecipe(burger));

apps/202-meal-planner-indirect-input-starter/src/app/meal-planner/meal-planner.spec.ts

@@ -9,7 +9,7 @@ import {
 } from './meal-repository.fake';
 
 describe(MealPlanner.name, () => {
-  it('should add recipe', async () => {
+  it('adds recipes', async () => {
     const { mealPlanner, burger, salad } = createMealPlanner();
 
     mealPlanner.addRecipe(burger);
@@ -21,15 +21,15 @@ describe(MealPlanner.name, () => {
     ]);
   });
 
-  it('should throw error if recipe is already present', () => {
+  it('throws error if recipe is already present', () => {
     const { mealPlanner, burgerDuplicate } = createMealPlannerWithBurger();
 
     expect(() => mealPlanner.addRecipe(burgerDuplicate)).toThrow(
       `Can't add recipe.`,
     );
   });
 
-  it('should add recipe to meal repository', () => {
+  it('adds recipe to meal repository', () => {
     const { mealPlanner, mealRepoFake, burger } = createMealPlanner();
 
     mealPlanner.addRecipe(burger);
@@ -42,7 +42,7 @@ describe(MealPlanner.name, () => {
   it.todo('🚧 should fetch recipes from meal repository');
 
   describe('recipes$', () => {
-    it('should emit empty array when no recipes', async () => {
+    it('emits empty array when no recipes', async () => {
       const { mealPlanner } = createMealPlanner();
 
       using observer = observe(mealPlanner.recipes$);
@@ -51,7 +51,7 @@ describe(MealPlanner.name, () => {
       expect(observer.next).toHaveBeenCalledWith([]);
     });
 
-    it('should emit recipes when added', () => {
+    it('emits recipes when added', () => {
       const { mealPlanner, burger, salad } = createMealPlanner();
 
       using observer = observe(mealPlanner.recipes$);
@@ -73,7 +73,7 @@ describe(MealPlanner.name, () => {
   });
 
   describe('watchCanAddRecipe()', () => {
-    it('should instantly emit if recipe can be added', () => {
+    it('emits instantly if recipe can be added', () => {
       const { mealPlanner, burger } = createMealPlanner();
 
       using observer = observe(mealPlanner.watchCanAddRecipe(burger));