search

rpm-software-management / spec-cleaner

spec-cleaner
BSD 3-Clause "New" or "Revised" License
28 stars 34 forks source link

Enhance/Correct test-case for pattern definitions #225

Closed DimStar77 closed 6 years ago

DimStar77 commented 6 years ago

The out/patterns is about what I'd expect it to be once corrected.

scarabeusiv commented 6 years ago

@simotek could you pretty please check if the resulting output is pleasing to your eyes? :)

simotek commented 6 years ago

I think the Requires: / Recommends: for other patterns should come after the Provides and Obsoletes, the first case in the diff is right, but the second and third aren't

DimStar77 commented 6 years ago

Arguably - but I can live with it :) - will adjust the test case

DimStar77 commented 6 years ago

Updated the test case... I intentinally 'messed' the input file up a bit more - so the sorting algo can be better tested :)

scarabeusiv commented 6 years ago

Oky so let me merge this ;)