Page MenuHomeFreeBSD
Differential D40297

Add simple factor/primes regression tests
ClosedPublic
Actions

Authored by ngie on May 27 2023, 4:53 AM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Jan 24, 4:01 AM
Unknown Object (File)
Dec 1 2024, 12:17 PM
Unknown Object (File)
Nov 20 2024, 5:44 AM
Unknown Object (File)
Oct 23 2024, 10:07 AM
Unknown Object (File)
Oct 21 2024, 2:16 AM
Unknown Object (File)
Oct 6 2024, 2:14 PM
Unknown Object (File)
Oct 4 2024, 11:34 PM
Unknown Object (File)
Oct 4 2024, 2:47 PM
View All 64 Files
Subscribers
imp

Details

Reviewers
emaste
Group Reviewers
tests
Commits
rGc60be9ea6b1e: Add simple factor/primes regression tests
Summary

This will help ensure that the change following this
one to support OpenSSL 3 is sane.

MFC after: 1 week

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Skipped
Unit
Tests Skipped
Build Status
Buildable 51731
Build 48622: arc lint + arc unit

Event Timeline

ngie created this revision.May 27 2023, 4:53 AM
Herald added a subscriber: imp. · View Herald TranscriptMay 27 2023, 4:53 AM
ngie requested review of this revision.May 27 2023, 4:53 AM
Harbormaster completed remote builds in B51731: Diff 122507.May 27 2023, 4:53 AM
ngie edited the summary of this revision. (Show Details)May 27 2023, 4:54 AM
ngie added a reviewer: tests.
emaste added inline comments.May 27 2023, 1:27 PM
usr.bin/factor/tests/factor_tests.sh
5

"All rights reserved." has been removed from our standard copyright template. It's unnecessary (but causes no harm if it's there).

35

it seems like factor should reject non-integer args instead

emaste accepted this revision.May 27 2023, 4:21 PM
This revision is now accepted and ready to land.May 27 2023, 4:21 PM
emaste added a comment.May 27 2023, 4:22 PM

anyhow, I think it's reasonable to commit this version and update if/when we change factor to disallow non-integer args

ngie added inline comments.May 27 2023, 6:05 PM
usr.bin/factor/tests/factor_tests.sh
35

Agreed. The fact that factor/primes support non-base 10 values, e.g., a, z, etc, is unexpected/undesirable.

Closed by commit rGc60be9ea6b1e: Add simple factor/primes regression tests (authored by ngie). · Explain WhyMay 27 2023, 6:12 PM
This revision was automatically updated to reflect the committed changes.
ngie added a commit: rGc60be9ea6b1e: Add simple factor/primes regression tests.

Revision Contents
Changeset List

PathSize
etc/
mtree/
2 lines
usr.bin/
factor/
3 lines
tests/
4 lines
86 lines
76 lines

Diff 122507

View Options

etc/mtree/BSD.tests.dist

Loading...
View Options

usr.bin/factor/Makefile

Loading...
View Options

usr.bin/factor/tests/Makefile

Loading...
View Options

usr.bin/factor/tests/factor_tests.sh

Loading...
View Options

usr.bin/factor/tests/primes_tests.sh

Loading...