Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Store hashes for generated input #210

Open
klenin opened this issue May 27, 2017 · 1 comment
Open

Store hashes for generated input #210

klenin opened this issue May 27, 2017 · 1 comment

Comments

@klenin
Copy link
Owner

klenin commented May 27, 2017 •
edited
Loading

Add hash attribute to Test\In tag in package.
Read and store hash with algorithm prefix, use sha1 by default.

Add field tests.in_file_hash VARCHAR(100).

After package upload, check constant input files against hashes.
After test generation, if hash is NULL, store it, otherwise compare and UH if they are not equal.
@klenin
Copy link
Owner Author

klenin commented Feb 21, 2018

Also, consider CRLF normalization. Problem flag? Always normalize to LF?

AleksandraZh added a commit to AleksandraZh/cats-main that referenced this issue Mar 2, 2018
@AleksandraZh
Store hashes for tests (klenin#210)
44c0586
AleksandraZh added a commit to AleksandraZh/cats-main that referenced this issue Mar 2, 2018
@AleksandraZh
Store hashes for tests (klenin#210)
ef04695
AleksandraZh added a commit to AleksandraZh/cats-main that referenced this issue Mar 2, 2018
@AleksandraZh
Store hashes for tests (klenin#210)
4442889
AleksandraZh added a commit to AleksandraZh/cats-main that referenced this issue Mar 3, 2018
@AleksandraZh
Store hashes for tests. See klenin#210
28e42d2
AleksandraZh added a commit to AleksandraZh/cats-main that referenced this issue Mar 3, 2018
@AleksandraZh
Store hashes for tests. See klenin#210
c9f2289
AleksandraZh added a commit to AleksandraZh/cats-main that referenced this issue Mar 24, 2018
@AleksandraZh
Store hashes for tests. See klenin#210
3821e8d
klenin added a commit that referenced this issue Apr 8, 2018
@klenin
Merge pull request #244 from AleksandraZh/master
c92ebf9 
Store hashes for tests (#210)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@klenin