Skip to content

Add "event" test fixture factory#2

Closed
SergioSim wants to merge 4 commits into
openfun:masterfrom
SergioSim:master
Closed

Add "event" test fixture factory#2
SergioSim wants to merge 4 commits into
openfun:masterfrom
SergioSim:master

Conversation

@SergioSim

@SergioSim SergioSim commented Feb 27, 2020

Copy link
Copy Markdown
Collaborator

Purpose

  • Document the different events that are logged as "tracking logs" in the LMS to understand how, were and when things get logged
  • A event test fixture factory could help to test the function of the GELFParser

Proposal

  • Implementation of the event test fixture factory
  • Events documentation

Comment thread tests/fixtures/logs.py Outdated
Comment thread tests/fixtures/logs.py Outdated
Comment thread tests/fixtures/logs.py Outdated
Comment thread tests/test_parsers.py Outdated
Comment thread tests/fixtures/logs.py Outdated
@jmaupetit

Copy link
Copy Markdown
Contributor

Also, can you provide a description to this PR using the following outline:

Purpose

Describe the problem here.

Proposal

Describe the implemented solution here.

@jmaupetit jmaupetit changed the title ✅(tests) do not merge - work in progress on log gene… Add "event" test fixture factory Feb 27, 2020
@SergioSim
SergioSim force-pushed the master branch 2 times, most recently from e7dbe5f to 4e07483 Compare March 5, 2020 08:48
@SergioSim
SergioSim force-pushed the master branch 6 times, most recently from 37543d6 to f1cbd6b Compare March 9, 2020 15:23
Comment thread setup.cfg Outdated
Comment thread tests/conftest.py Outdated
@SergioSim
SergioSim force-pushed the master branch 15 times, most recently from a32e21a to 825c871 Compare March 13, 2020 07:48
adding documenation for the different events (Server event/ browser
event/ triggered event ongoing)
adding log generator for testing purposes
@SergioSim
SergioSim force-pushed the master branch 6 times, most recently from c08be39 to e80356a Compare April 23, 2020 11:39
@SergioSim
SergioSim force-pushed the master branch 8 times, most recently from 0e425ec to 0eb65eb Compare May 14, 2020 08:48
Comment thread .vscode/settings.json Outdated
Comment thread tests/fixtures/event/base.py
Comment thread tests/fixtures/event/base.py Outdated
Comment thread .gitignore Outdated
        first attempt with the dataclass / moving fields from base to parent
@SergioSim

Copy link
Copy Markdown
Collaborator Author

Created new pull request from a feature branch, see #3

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants