London | 26-ITP-Jan | Maryanne Mosonik | Sprint 2 | Module Data Groups

[Maryanne-K]

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

Changelist

Corrected and completed the implementations and jest test cases.

[cjyuan]

test.todo(" ... ") is just a placeholder to serve as a reminder for "what needs to be done".
In this exercise, that means "implementing the corresponding Jest tests described in the comments."

Can you replace them (in all .test.js files) with actual Jest tests?

[cjyuan]

The requirement on line 4 has not yet been met:

Each ingredient should be logged on a new line

[cjyuan]

Consider the following two approaches for determining if an object contains a property:

  let obj = {}, propertyName = "toString";
  console.log( propertyName in obj );                // true
  console.log( Object.hasOwn(obj, propertyName) );   // false

Which of these approaches suits your needs better?
For more info, you can look up JS "in" operator vs Object.hasOwn.

[cjyuan]

Note: (No change required)

• In real query string, both key and value are percent-encoded or URL encoded in the URL.
  For example, the string "5%" is encoded as "5%25". So to get the actual value of "5%25"
  (whether it is a key or value in the query string), we need to call a function to decode it.

• You can also explore the URLSearchParams API.

[cjyuan]

Does the following function call returns the value you expect?

tally(["toString", "toString"]);

Suggestion: Look up an approach to create an empty object with no inherited properties.