Skip to content

Mock mongo event tracking backend during tests - #938

Merged
wedaly merged 1 commit into
openedx:masterfrom
rocha:mock-mongo-on-event-tracking-tests
Sep 12, 2013
Merged

Mock mongo event tracking backend during tests#938
wedaly merged 1 commit into
openedx:masterfrom
rocha:mock-mongo-on-event-tracking-tests

Conversation

@rocha

@rocha rocha commented Sep 11, 2013

Copy link
Copy Markdown
Contributor

@wedaly @mulby review please

@wedaly

wedaly commented Sep 12, 2013

Copy link
Copy Markdown
Contributor

👍 (sorry I didn't see this earlier this morning)

@rocha

rocha commented Sep 12, 2013

Copy link
Copy Markdown
Contributor Author

@mulby fixed the patcher

@rocha

rocha commented Sep 12, 2013

Copy link
Copy Markdown
Contributor Author

@mulby good to merge?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You should just add this as a .cleanup

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed

@mulby

mulby commented Sep 12, 2013

Copy link
Copy Markdown
Contributor

👍

1 similar comment
@cpennington

Copy link
Copy Markdown
Contributor

👍

@wedaly

wedaly commented Sep 12, 2013

Copy link
Copy Markdown
Contributor

@rocha Merging this

wedaly pushed a commit that referenced this pull request Sep 12, 2013
Mock mongo event tracking backend during tests
@wedaly
wedaly merged commit 2677598 into openedx:master Sep 12, 2013
@rocha
rocha deleted the mock-mongo-on-event-tracking-tests branch September 12, 2013 19:36
bradenmacdonald pushed a commit to open-craft/openedx-platform that referenced this pull request Oct 13, 2017
caesar2164 pushed a commit to caesar2164/edx-platform that referenced this pull request Oct 7, 2019
* Commits:
  Update reference to undefined variable in nonregistered
DanielVZ96 pushed a commit to open-craft/openedx-platform that referenced this pull request Jan 31, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants