Pantry removal sorting

Group Phase Implementation
1

Hi!

I noticed that our pantry removal does not sort alphabetically, but instead in insertion/purchase order.

The spec says (on page 31):

he simulation has to log a message containing the id of the restaurant and the amount, unit and name of each ingredient that was removed from the pantry. The order is by ascending alphabetic order of the name of the ingredient.

However, our test we created previously, that assumed insertion order instead of alphabetical order, passed against the reference implementation.

To me, this appears to be a bug in the reference implementation.

Also, actually sorting alphabetically broke 5 component tests and at least 2 full tests for us…

2

Hi,

the correct order of sorting the pantry logs is alphabetically by the ingredient names. I guess something else went wrong there.

You might want to join the office hour to discuss you test further.

2 Likes
3

For the procuring logs, it works. For the removal logs, it broke tests. I’ll come to the office hour.

1 Like