Brussels | ITP-2025-1 | Daniela Fajardo | Sprint 2 | Implement and Interpret

[ldfajardo10-tech]

Learners, PR Template

Self checklist

• I have titled my PR with Region | Cohort | FirstName LastName | Sprint | Assignment Title
• My changes meet the requirements of the task
• I have tested my changes
• My changes follow the style guide

Implement & Interpret Exercises:

• Contains
• Lookup
• Tally
• QueryStrings
• Invert

I forget to create a new branch for the interpret/invert exercise so I pushed in this branch.

[fgsumer]

@ldfajardo10-tech

Overall, nice work.

There are some changes needed.
Can you update your code based on the feedback below, please?

• For tally(), createLookup() and contains() there are just console.log()’s. These are not formal tests. You should write tests using a test framework like test(...) and expect(...)

• In queryString.js, queryParams is used but not declared

• In invert.js, toBe() is used to compare references. It should be toEqual().

• In invert.js, a and b are not defined in test scope. This is how the test look like :

     test("inverts object with numeric values", () => {
        expect(invert({ a: 1, b: 2 })).toEqual({ "1": "a", "2": "b" });
     });

[ldfajardo10-tech]

In querystring : I dont understand why is not showing up the first lines of code in the commit. when I checked directly into the folder the file have all the function and the queryparams variable.