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

[REF] [Import] [Trivial] Minor test fixes #23341

Merged
merged 1 commit into from
May 3, 2022
Merged

[REF] [Import] [Trivial] Minor test fixes #23341

merged 1 commit into from
May 3, 2022

Conversation

eileenmcnaughton
Copy link
Contributor

This just fixes some of the import parser tests to be more
realistic - notably ensuring the userJobID is set - which
is now required, and making the params for
setting that up more flexible within the test class

@eileenmcnaughton
[REF] [Import] [Trivial] Minor test fixes
017bd22 
This just fixes some of the import parser tests to be more
realistic - notably ensuring the userJobID is set - which
is now required, and making the params for
setting that up more flexible within the test class
@civibot
Copy link

civibot bot commented May 3, 2022

(Standard links)

@civibot civibot bot added the master label May 3, 2022
@eileenmcnaughton eileenmcnaughton merged commit d9d0e6a into civicrm:master May 3, 2022
@eileenmcnaughton eileenmcnaughton deleted the import_test branch May 3, 2022 02:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
master merge on pass
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@eileenmcnaughton @seamuslee001