While working on

#2808063: LibraryDiscoveryParser::buildByExtension() doesn't validate that extensions exist

I noticed a little blemish - of no consequence, that I nevertheless want to take out.

One of the test inserts an unwanted extra parameter.

CommentFileSizeAuthor
#2 buildBy-2822665-2.patch788 bytesmartin107

Comments

martin107 created an issue. See original summary.

martin107’s picture

martin107 commented
Title: LibraryDiscoveryParserTest::testBuilldByExtensionSimple() passes an unwanted paramtere. » LibraryDiscoveryParserTest::testBuilldByExtensionSimple() passes an unwanted parameter
Assigned: martin107 » Unassigned
Status: Active » Needs review
StatusFileSize
new buildBy-2822665-2.patch788 bytes

patch.

chi’s picture

chi commented
Version: 8.3.x-dev » 8.2.x-dev
Status: Needs review » Reviewed & tested by the community

I suppose bugs should be fixed in 8.2.x.

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
commented
Status: Reviewed & tested by the community » Fixed

Committed and pushed 4920ac0 to 8.3.x and 1504e62 to 8.2.x. Thanks!

  • alexpott committed 4920ac0 on 8.3.x 
    Issue #2822665 by martin107: LibraryDiscoveryParserTest::...

  • alexpott committed 1504e62 on 8.2.x 
    Issue #2822665 by martin107: LibraryDiscoveryParserTest::...

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.