[Bug 1869719] Review Request: rubygem-image_processing - High-level wrapper for processing images for the web with ImageMagick or libvips

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



https://bugzilla.redhat.com/show_bug.cgi?id=1869719

Pavel Valena <pvalena@xxxxxxxxxx> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|ASSIGNED                    |MODIFIED
   Fixed In Version|                            |rubygem-image_processing-1.
                   |                            |11.0-1.fc34



--- Comment #9 from Pavel Valena <pvalena@xxxxxxxxxx> ---
""" the `skip` won't execute the following asserts """

Are you sure about that? I've always thought it's the other way around:

```
# SKIP:
DEBUG: # Running:
DEBUG:
SSSSSSSSSSSSSSS..S.SSSSSSSS.....S..S...SSS..SSS...S.SS..S..S..................S.S...S.S.
DEBUG: Finished in 3.638647s, 24.1848 runs/s, 20.3372 assertions/s.
DEBUG: 88 runs, 74 assertions, 0 failures, 0 errors, 41 skips

# RETURN:
DEBUG: + ruby -Ilib:test -e 'Dir.glob "./test/**/*_test.rb", &method(:require)'
DEBUG: Run options: --seed 39129
DEBUG: # Running:
DEBUG:
........................................................................................
DEBUG: Finished in 4.193257s, 20.9861 runs/s, 19.5552 assertions/s.
DEBUG: 88 runs, 82 assertions, 0 failures, 0 errors, 0 skips
```

And with the same test-set, but vips enabled:

```
DEBUG: Run options: --seed 32767
DEBUG: # Running:
DEBUG:
........................................................................................
DEBUG: Finished in 5.651056s, 15.5723 runs/s, 26.5437 assertions/s.
DEBUG: 88 runs, 150 assertions, 0 failures, 0 errors, 0 skips
```

aaand ... now I'm even more confused. So the return has (8) more assertions,
and `skip` does not count show all assertions skipped (return hides those,
obviously)?


-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
_______________________________________________
package-review mailing list -- package-review@xxxxxxxxxxxxxxxxxxxxxxx
To unsubscribe send an email to package-review-leave@xxxxxxxxxxxxxxxxxxxxxxx
Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: https://lists.fedoraproject.org/archives/list/package-review@xxxxxxxxxxxxxxxxxxxxxxx




[Index of Archives]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite Conditions]     [KDE Users]

  Powered by Linux