-
Notifications
You must be signed in to change notification settings - Fork 0
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
- Loading branch information
1 parent
968691b
commit 1c61d8c
Showing
52 changed files
with
2,724 additions
and
0 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,41 @@ | ||
# This is a Code Annotations automatically-generated Django model safelist file. | ||
# These models must be annotated as follows in order to be counted in the coverage report. | ||
# See https://code-annotations.readthedocs.io/en/latest/safelist.html for more information. | ||
# | ||
# fake_app_1.FakeModelName: | ||
# ".. no_pii:": "This model has no PII" | ||
# fake_app_2.FakeModel2: | ||
# ".. choice_annotation:": foo, bar, baz | ||
|
||
admin.LogEntry: | ||
".. no_pii:": "This model has no PII" | ||
auth.Group: | ||
".. no_pii:": "This model has no PII" | ||
auth.Permission: | ||
".. no_pii:": "This model has no PII" | ||
auth.User: | ||
".. pii": "This model minimally contains a username, password, and email" | ||
".. pii_types": "username, email_address, password" | ||
".. pii_retirement": "consumer_api" | ||
contenttypes.ContentType: | ||
".. no_pii:": "This model has no PII" | ||
sessions.Session: | ||
".. no_pii:": "This model has no PII" | ||
social_django.Association: | ||
".. no_pii:": "This model has no PII" | ||
social_django.Code: | ||
".. pii:": "Email address" | ||
".. pii_types:": other | ||
".. pii_retirement:": local_api | ||
social_django.Nonce: | ||
".. no_pii:": "This model has no PII" | ||
social_django.Partial: | ||
".. no_pii:": "This model has no PII" | ||
social_django.UserSocialAuth: | ||
".. no_pii:": "This model has no PII" | ||
waffle.Flag: | ||
".. no_pii:": "This model has no PII" | ||
waffle.Sample: | ||
".. no_pii:": "This model has no PII" | ||
waffle.Switch: | ||
".. no_pii:": "This model has no PII" |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,100 @@ | ||
# *************************** | ||
# ** DO NOT EDIT THIS FILE ** | ||
# *************************** | ||
# | ||
# This file was generated by edx-lint: https://github.com/openedx/edx-lint | ||
# | ||
# If you want to change this file, you have two choices, depending on whether | ||
# you want to make a local change that applies only to this repo, or whether | ||
# you want to make a central change that applies to all repos using edx-lint. | ||
# | ||
# Note: If your .editorconfig file is simply out-of-date relative to the latest | ||
# .editorconfig in edx-lint, ensure you have the latest edx-lint installed | ||
# and then follow the steps for a "LOCAL CHANGE". | ||
# | ||
# LOCAL CHANGE: | ||
# | ||
# 1. Edit the local .editorconfig_tweaks file to add changes just to this | ||
# repo's file. | ||
# | ||
# 2. Run: | ||
# | ||
# $ edx_lint write .editorconfig | ||
# | ||
# 3. This will modify the local file. Submit a pull request to get it | ||
# checked in so that others will benefit. | ||
# | ||
# | ||
# CENTRAL CHANGE: | ||
# | ||
# 1. Edit the .editorconfig file in the edx-lint repo at | ||
# https://github.com/openedx/edx-lint/blob/master/edx_lint/files/.editorconfig | ||
# | ||
# 2. install the updated version of edx-lint (in edx-lint): | ||
# | ||
# $ pip install . | ||
# | ||
# 3. Run (in edx-lint): | ||
# | ||
# $ edx_lint write .editorconfig | ||
# | ||
# 4. Make a new version of edx_lint, submit and review a pull request with the | ||
# .editorconfig update, and after merging, update the edx-lint version and | ||
# publish the new version. | ||
# | ||
# 5. In your local repo, install the newer version of edx-lint. | ||
# | ||
# 6. Run: | ||
# | ||
# $ edx_lint write .editorconfig | ||
# | ||
# 7. This will modify the local file. Submit a pull request to get it | ||
# checked in so that others will benefit. | ||
# | ||
# | ||
# | ||
# | ||
# | ||
# STAY AWAY FROM THIS FILE! | ||
# | ||
# | ||
# | ||
# | ||
# | ||
# SERIOUSLY. | ||
# | ||
# ------------------------------ | ||
# Generated by edx-lint version: 5.2.5 | ||
# ------------------------------ | ||
[*] | ||
end_of_line = lf | ||
insert_final_newline = true | ||
charset = utf-8 | ||
indent_style = space | ||
indent_size = 4 | ||
max_line_length = 120 | ||
trim_trailing_whitespace = true | ||
|
||
[{Makefile, *.mk}] | ||
indent_style = tab | ||
indent_size = 8 | ||
|
||
[*.{yml,yaml,json}] | ||
indent_size = 2 | ||
|
||
[*.js] | ||
indent_size = 2 | ||
|
||
[*.diff] | ||
trim_trailing_whitespace = false | ||
|
||
[.git/*] | ||
trim_trailing_whitespace = false | ||
|
||
[COMMIT_EDITMSG] | ||
max_line_length = 72 | ||
|
||
[*.rst] | ||
max_line_length = 79 | ||
|
||
# f2f02689fced7a2e0c62c2f9803184114dc2ae4b |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,39 @@ | ||
**Description:** Describe in a couple of sentences what this PR adds | ||
|
||
**JIRA:** Link to JIRA ticket | ||
|
||
**Dependencies:** dependencies on other outstanding PRs, issues, etc. | ||
|
||
**Merge deadline:** List merge deadline (if any) | ||
|
||
**Installation instructions:** List any non-trivial installation | ||
instructions. | ||
|
||
**Testing instructions:** | ||
|
||
1. Open page A | ||
2. Do thing B | ||
3. Expect C to happen | ||
4. If D happened instead - check failed. | ||
|
||
**Reviewers:** | ||
- [ ] tag reviewer | ||
- [ ] tag reviewer | ||
|
||
**Merge checklist:** | ||
- [ ] All reviewers approved | ||
- [ ] CI build is green | ||
- [ ] Version bumped | ||
- [ ] Changelog record added | ||
- [ ] Documentation updated (not only docstrings) | ||
- [ ] Commits are squashed | ||
|
||
**Post merge:** | ||
- [ ] Create a tag | ||
- [ ] Check new version is pushed to PyPI after tag-triggered build is | ||
finished. | ||
- [ ] Delete working branch (if not needed anymore) | ||
|
||
**Author concerns:** List any concerns about this PR - inelegant | ||
solutions, hacks, quick-and-dirty implementations, concerns about | ||
migrations, etc. |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,10 @@ | ||
# Run commitlint on the commit messages in a pull request. | ||
|
||
name: Lint Commit Messages | ||
|
||
on: | ||
- pull_request | ||
|
||
jobs: | ||
commitlint: | ||
uses: openedx/.github/.github/workflows/commitlint.yml@master |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,100 @@ | ||
name: Integration Tests | ||
|
||
on: | ||
pull_request: | ||
push: | ||
branches: [main] | ||
|
||
concurrency: | ||
group: "${{ github.workflow }}-${{ github.ref }}" | ||
cancel-in-progress: true | ||
|
||
jobs: | ||
tests: | ||
name: ${{ matrix.edx-platform.branch }} | ||
runs-on: ubuntu-20.04 | ||
strategy: | ||
fail-fast: false | ||
matrix: | ||
edx-platform: | ||
- branch: opencraft-release/redwood.1 | ||
remote: open-craft | ||
python_version: 3.11 | ||
- branch: open-release/sumac.master | ||
remote: openedx | ||
python_version: 3.11 | ||
- branch: master | ||
remote: openedx | ||
python_version: 3.11 | ||
env: | ||
EDX_PLATFORM_PATH: ./edx/app/edxapp/edx-platform | ||
PLUGIN_PATH: ./edx/src/openedx-course-enrollment-audit | ||
|
||
steps: | ||
- name: Create directory structure | ||
run: mkdir -p ${{ env.EDX_PLATFORM_PATH }} ${{ env.PLUGIN_PATH }} | ||
|
||
- name: Checkout edx-platform repository | ||
uses: actions/checkout@v4 | ||
with: | ||
repository: ${{ matrix.edx-platform.remote }}/edx-platform | ||
ref: ${{ matrix.edx-platform.branch }} | ||
path: ${{ env.EDX_PLATFORM_PATH }} | ||
|
||
- name: install edx-platform required packages | ||
run: sudo apt-get update && sudo apt-get install libxmlsec1-dev lynx | ||
|
||
- name: Start MongoDB | ||
uses: supercharge/[email protected] | ||
with: | ||
mongodb-version: 7.0 | ||
|
||
- name: Setup Python | ||
uses: actions/setup-python@v4 | ||
with: | ||
python-version: ${{ matrix.edx-platform.python_version }} | ||
|
||
- name: Get pip cache dir | ||
id: pip-cache-dir | ||
run: echo "dir=$(pip cache dir)" >> $GITHUB_OUTPUT | ||
|
||
- name: Cache pip dependencies | ||
id: cache-dependencies | ||
uses: actions/cache@v3 | ||
with: | ||
path: ${{ steps.pip-cache-dir.outputs.dir }} | ||
key: ${{ runner.os }}-pip-${{ hashFiles('requirements/edx/testing.txt') }} | ||
restore-keys: ${{ runner.os }}-pip- | ||
|
||
- name: Install edx-platform required Python dependencies | ||
env: | ||
PIP_SRC: ${{ runner.temp }} | ||
working-directory: ${{ env.EDX_PLATFORM_PATH }} | ||
run: make test-requirements | ||
|
||
- name: Checkout plugin repo | ||
uses: actions/checkout@v4 | ||
with: | ||
path: ${{ env.PLUGIN_PATH }} | ||
|
||
- name: Install plugin | ||
working-directory: ${{ env.PLUGIN_PATH }} | ||
run: pip install -e . | ||
|
||
# FIXME: The test config fails with: `settings.DATABASES is improperly configured. Please supply the NAME value.` | ||
# The devstack config fails with `Set the LMS_CFG env variable`. | ||
# - name: Check if migrations are up-to-date | ||
# working-directory: ${{ env.PLUGIN_PATH }} | ||
# run: make test_migrations | ||
|
||
- name: Run plugin integration tests | ||
working-directory: ${{ env.PLUGIN_PATH }} | ||
run: make test_integration | ||
|
||
- name: Run coverage | ||
if: matrix.edx-platform.branch == 'opencraft-release/redwood.1' | ||
uses: codecov/codecov-action@v5 | ||
with: | ||
working-directory: ${{ env.PLUGIN_PATH }} | ||
token: ${{ secrets.CODECOV_TOKEN }} | ||
fail_ci_if_error: true |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,17 @@ | ||
name: Lint | ||
|
||
on: | ||
push: | ||
branches: [main] | ||
pull_request: | ||
branches: | ||
- '**' | ||
|
||
jobs: | ||
lint: | ||
runs-on: ubuntu-latest | ||
steps: | ||
- uses: actions/checkout@v4 | ||
- uses: astral-sh/ruff-action@v1 | ||
with: | ||
args: "format --check" |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,30 @@ | ||
name: Publish package to PyPi | ||
|
||
on: | ||
release: | ||
types: [published] | ||
|
||
jobs: | ||
|
||
push: | ||
runs-on: ubuntu-20.04 | ||
|
||
steps: | ||
- name: Checkout | ||
uses: actions/checkout@v3 | ||
- name: setup python | ||
uses: actions/setup-python@v4 | ||
with: | ||
python-version: 3.11 | ||
|
||
- name: Install pip | ||
run: pip install -r requirements/pip.txt | ||
|
||
- name: Build package | ||
run: python setup.py sdist bdist_wheel | ||
|
||
- name: Publish to PyPi | ||
uses: pypa/gh-action-pypi-publish@release/v1 | ||
with: | ||
user: __token__ | ||
password: ${{ secrets.PYPI_UPLOAD_TOKEN }} |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,15 @@ | ||
name: Quality | ||
|
||
on: | ||
push: | ||
branches: [main] | ||
pull_request: | ||
branches: | ||
- '**' | ||
|
||
jobs: | ||
quality: | ||
runs-on: ubuntu-latest | ||
steps: | ||
- uses: actions/checkout@v4 | ||
- uses: astral-sh/ruff-action@v1 |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,36 @@ | ||
name: Unit Tests | ||
|
||
on: | ||
push: | ||
branches: [main] | ||
pull_request: | ||
branches: | ||
- '**' | ||
|
||
jobs: | ||
run_tests: | ||
name: tests | ||
runs-on: ${{ matrix.os }} | ||
strategy: | ||
matrix: | ||
os: [ubuntu-20.04] | ||
python-version: ['3.11'] | ||
toxenv: [pii_check, package] | ||
|
||
steps: | ||
- uses: actions/checkout@v3 | ||
- name: setup python | ||
uses: actions/setup-python@v4 | ||
with: | ||
python-version: ${{ matrix.python-version }} | ||
|
||
- name: Install pip | ||
run: pip install -r requirements/pip.txt | ||
|
||
- name: Install Dependencies | ||
run: pip install -r requirements/ci.txt | ||
|
||
- name: Run Tests | ||
env: | ||
TOXENV: ${{ matrix.toxenv }} | ||
run: tox |
Oops, something went wrong.