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

Add tests for tpf_fasta #31

Merged
merged 26 commits into from
Aug 15, 2024
Merged

Add tests for tpf_fasta #31

merged 26 commits into from
Aug 15, 2024

Conversation

stevieing
Copy link
Contributor

  • Add tests for check_orientation
  • Add tests for get_uniques
  • Make various things publicly available so we can test them

stevieing and others added 26 commits June 18, 2024 10:56
@stevieing
Add tests for check_orientation and get_uniques. publicly expose thin…
e64a20a 
…gs we don't want to.
@stevieing
Refactor code to remove extra line and add indent in tests
44c1109
@stevieing
Add test for get_subset_of_tpfs
1ebaa8d
@dasunpubudumal
Adding tests for tpf_fasta.parse_seq
8377ad2
@dasunpubudumal
Add tests for tpf_fasta.parse_seq
20b024a
@dasunpubudumal
[skip ci] Test documentation
6a93579
@dasunpubudumal
feat: Add test data files for iyAndFlav1 and yaml
c73b30a
@dasunpubudumal
Refactor code to remove unused functions and improve code readability
29d3f0a
@dasunpubudumal
Remove unused test data files and update YAML configuration
94794e4
@dasunpubudumal
Refactor code to remove unused functions and improve code readability
4c2535c
@dasunpubudumal
Merge branch 'dev' into 22-create-tests-for-tpf_fasta
2ab60d3
@dasunpubudumal
Refactor code to remove unused functions and improve code readability
20082c5
@stevieing
Created test for parse_tpf.
9b8f45d 
Create comments for how the save_to_fasta function is working.
@dasunpubudumal
Updating test data files from dev branch
ff4ac76
@dasunpubudumal
Writing tests for save_to_fasta function (and creating an assertion f…
eb86c7f 
…ile tiny_test.debug.txt)
@dasunpubudumal
Starting off check_curate_fasta function
b4e31be
@dasunpubudumal
Refactor code to remove unused functions and improve code readability
dca8720
@dasunpubudumal
Refactor code to use 'util' module for file comparison in tpf_fasta.r…
fc92fe3 
…s test
@dasunpubudumal
[skip ci] Adding docstrings for the test utility function
3d64a4f
@dasunpubudumal
chore: Add pattern matching to extract bool result in are_files_ident…
318caa5 
…ical function
@DLBPointon
Small changes to check_curate_fasta
2b040d9
@DLBPointon
Forgot to make it raw
2ff346d
@DLBPointon
Changed structure of test and input strings
7fbeea5
@DLBPointon
small changes
d6b4826
@stevieing
Merge pull request #41 from Rust-Wellcome/22-changes
f5771f7 
22 changes
@dasunpubudumal
Merge remote-tracking branch 'origin/dev' into 22-create-tests-for-tp…
46cd2ce 
…f_fasta

# Conflicts:
#	Cargo.lock
#	Cargo.toml
#	src/yaml_validator.rs
@DLBPointon DLBPointon merged commit 63b3f36 into dev Aug 15, 2024
5 checks passed
@stevieing stevieing linked an issue Aug 15, 2024 that may be closed by this pull request
Create tests for tpf_fasta #22
Closed
@DLBPointon DLBPointon deleted the 22-create-tests-for-tpf_fasta branch August 16, 2024 11:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@DLBPointon DLBPointon DLBPointon approved these changes

@dasunpubudumal dasunpubudumal Awaiting requested review from dasunpubudumal

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Create tests for tpf_fasta
3 participants
@stevieing @DLBPointon @dasunpubudumal