[GH-ISSUE #55] Modify tests to check for duplicate entries #311

Closed
opened 2026-03-14 01:13:02 +03:00 by kerem · 3 comments
Owner

Originally created by @silversword411 on GitHub (Jun 23, 2022).
Original GitHub issue: https://github.com/amidaware/community-scripts/issues/55

See:

github.com/amidaware/community-scripts@eaf62b855c/community_scripts.json (L578)

and

github.com/amidaware/community-scripts@eaf62b855c/community_scripts.json (L582)

Should probably make sure all fields aren't duplicated

Originally created by @silversword411 on GitHub (Jun 23, 2022). Original GitHub issue: https://github.com/amidaware/community-scripts/issues/55 See: https://github.com/amidaware/community-scripts/blob/eaf62b855c3994de5296e40d683fc803f0528dd0/community_scripts.json#L578 and https://github.com/amidaware/community-scripts/blob/eaf62b855c3994de5296e40d683fc803f0528dd0/community_scripts.json#L582 Should probably make sure all fields aren't duplicated
kerem closed this issue 2026-03-14 01:13:07 +03:00
Author
Owner

@silversword411 commented on GitHub (Jun 23, 2022):

@wh1te909 ...did we try doing this already and couldn't get it to work?

<!-- gh-comment-id:1164649956 --> @silversword411 commented on GitHub (Jun 23, 2022): @wh1te909 ...did we try doing this already and couldn't get it to work?
Author
Owner

@wh1te909 commented on GitHub (Jun 23, 2022):

python does not allow dicts to have duplicate keys so when it's imported, the duplicate is removed automatically, but this can lead to unexpected behavior because we won't know which line was removed. I have just addded a check to catch duplicates

<!-- gh-comment-id:1164689177 --> @wh1te909 commented on GitHub (Jun 23, 2022): python does not allow dicts to have duplicate keys so when it's imported, the duplicate is removed automatically, but this can lead to unexpected behavior because we won't know which line was removed. I have just addded a check to catch duplicates
Author
Owner

@wh1te909 commented on GitHub (Jun 23, 2022):

https://github.com/amidaware/community-scripts/runs/7028877647?check_suite_focus=true shows it catching it

<!-- gh-comment-id:1164689895 --> @wh1te909 commented on GitHub (Jun 23, 2022): https://github.com/amidaware/community-scripts/runs/7028877647?check_suite_focus=true shows it catching it
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
starred/community-scripts#311
No description provided.