fix: issue with multiple imports leading to empty output

[tstirrat15]

Description

We had a report that a particular schema structure wasn't working for a user. This adds a test for that structure and fixes the issue.

The tl;dr: is that schema content was being dropped when a previously-encountered import was seen again, because we were returning instead of continueing.

Changes

• Fix log message
• Change return to continue
• Add test and fix test

Testing

Review. See that tests pass

[barakmich]

Yay new tests too

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 78.51%. Comparing base (9e85369) to head (6622ed0).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #2804      +/-   ##
==========================================
+ Coverage   78.51%   78.51%   +0.01%     
==========================================
  Files         458      458              
  Lines       44253    44253              
==========================================
+ Hits        34741    34743       +2     
- Misses       6693     6695       +2     
+ Partials     2819     2815       -4     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.