Merge branch 'master' of https://github.com/source-academy/backend into cas

Author: RichDom2185

.credo.exs

@@ -94,7 +94,7 @@
         {Credo.Check.Readability.SpaceAfterCommas},
         {Credo.Check.Refactor.DoubleBooleanNegation},
         {Credo.Check.Refactor.CondStatements},
-        {Credo.Check.Refactor.CyclomaticComplexity},
+        {Credo.Check.Refactor.CyclomaticComplexity, max_complexity: 10},
         {Credo.Check.Refactor.FunctionArity},
         {Credo.Check.Refactor.LongQuoteBlocks},
         {Credo.Check.Refactor.MatchInCondition},

.github/workflows/cd.yml

@@ -5,6 +5,7 @@ on:
       - stable
       - master
       - deploy
+      - deploy-stg
     paths:
       - "config/**"
       - "lib/**"
@@ -28,16 +29,16 @@ jobs:
       OTP_VERSION: 25.3.2
     steps:
       - uses: rlespinasse/[email protected]
-      - uses: actions/checkout@v2
+      - uses: actions/checkout@v4
       - name: Cache deps
-        uses: actions/cache@v2
+        uses: actions/cache@v4
         with:
           path: deps
           key: ${{ runner.os }}-mix-${{ hashFiles(format('{0}{1}', github.workspace, '/mix.lock')) }}
           restore-keys: |
             ${{ runner.os }}-mix-
       - name: Cache _build
-        uses: actions/cache@v2
+        uses: actions/cache@v4
         with:
           path: |
             _build

.github/workflows/ci.yml

@@ -37,16 +37,16 @@ jobs:
         # needed because the postgres container does not provide a healthcheck
         options: --health-cmd pg_isready --health-interval 10s --health-timeout 5s --health-retries 5
     steps:
-      - uses: actions/checkout@v2
+      - uses: actions/checkout@v4
       - name: Cache deps
-        uses: actions/cache@v2
+        uses: actions/cache@v4
         with:
           path: deps
           key: ${{ runner.os }}-mix-${{ hashFiles(format('{0}{1}', github.workspace, '/mix.lock')) }}
           restore-keys: |
             ${{ runner.os }}-mix-
       - name: Cache _build
-        uses: actions/cache@v2
+        uses: actions/cache@v4
         with:
           path: |
             _build

.gitignore

@@ -110,6 +110,10 @@ erl_crash.dump
 # VSCode Elixir language support
 .elixir_ls/
 
+# IDEs
+.idea
+.vscode
+
 .env
 
 # Generated lexer

.iex.exs

@@ -1,5 +1,5 @@
 import Ecto.Query
 alias Cadet.Repo
-alias Cadet.Accounts.User
+alias Cadet.Accounts.{User, Team, TeamMember}
 alias Cadet.Assessments.{Answer, Assessment, Question, Submission}
 alias Cadet.Courses.Group

README.md

@@ -62,6 +62,12 @@ It is probably okay to use a different version of PostgreSQL or Erlang/OTP, but
    $ sudo service postgresql restart
    ```
 
+   By default, the database is populated with 10 students and 5 assessments. Each student will have a submission to the corresponding submission. This can be changed in `priv/repo/seeds.exs` with the variables `number_of_students`, `number_of_assessments` and `number_of_questions`. Save the changes and run:
+
+   ```bash
+   $ mix ecto.reset
+   ```
+
 4. Run the server on your local machine
 
    ```bash
@@ -141,7 +147,7 @@ Where there is a conflict between the two, the first one (lexmag) shall be the o
 
 ## Entity-Relationship Diagram
 
-Generated with [DBeaver](https://dbeaver.io/) on 03 June 2022
+Generated with [DBeaver](https://dbeaver.io/) on 19 May 2024
 
 ![Entity-Relationship Diagram for cadet](schema.png)
 

config/cadet.exs.example

@@ -56,6 +56,15 @@ config :cadet,
     #      # You may need to write your own claim extractor for other providers
     #      claim_extractor: Cadet.Auth.Providers.CognitoClaimExtractor
     #    }},
+
+    # # Example SAML authentication with NUS Student IdP
+    # "test_saml" =>
+    #   {Cadet.Auth.Providers.SAML,
+    #    %{
+    #      assertion_extractor: Cadet.Auth.Providers.NusstuAssertionExtractor,
+    #      client_redirect_url: "http://cadet.frontend:8000/login/callback"
+    #    }},
+
     # # To use authentication with CAS
     # "cas" =>
     #   {Cadet.Auth.Providers.CAS,
@@ -147,3 +156,29 @@ config :cadet,
 # You may also want to change the timezone used for scheduled jobs
 # config :cadet, Cadet.Jobs.Scheduler,
 #   timezone: "Asia/Singapore",
+
+# # Additional configuration for SAML authentication
+# # For more details, see https://github.com/handnot2/samly
+# config :samly, Samly.Provider,
+#   idp_id_from: :path_segment,
+#   service_providers: [
+#     %{
+#       id: "source-academy-backend",
+#       certfile: "example_path/certfile.pem",
+#       keyfile: "example_path/keyfile.pem"
+#     }
+#   ],
+#   identity_providers: [
+#     %{
+#       id: "student",
+#       sp_id: "source-academy-backend",
+#       base_url: "https://example_backend/sso",
+#       metadata_file: "student_idp_metadata.xml"
+#     },
+#     %{
+#       id: "staff",
+#       sp_id: "source-academy-backend",
+#       base_url: "https://example_backend/sso",
+#       metadata_file: "staff_idp_metadata.xml"
+#     }
+#   ]

config/dev.secrets.exs.example

@@ -29,6 +29,15 @@ config :cadet,
     #       token_url: "https://github.com/login/oauth/access_token",
     #       user_api: "https://api.github.com/user"
     #    }},
+
+    # # Example SAML authentication with NUS Student IdP
+    # "test_saml" =>
+    #   {Cadet.Auth.Providers.SAML,
+    #    %{
+    #      assertion_extractor: Cadet.Auth.Providers.NusstuAssertionExtractor,
+    #      client_redirect_url: "http://cadet.frontend:8000/login/callback"
+    #    }},
+
     "test" =>
       {Cadet.Auth.Providers.Config,
        [
@@ -87,5 +96,39 @@ config :cadet,
     # ws_endpoint_address: "ws://hostname:port"
   ]
 
-config :sentry,
-  dsn: "https://public_key/sentry.io/somethingsomething"
+  config :openai,
+  # find it at https://platform.openai.com/account/api-keys
+  api_key: "the actual api key",
+  # For source academy deployment, leave this as empty string.Ingeneral could find it at https://platform.openai.com/account/org-settings under "Organization ID".
+  organization_key: "",
+  # optional, passed to [HTTPoison.Request](https://hexdocs.pm/httpoison/HTTPoison.Request.html) options
+  http_options: [recv_timeout: 170_0000]
+
+# config :sentry,
+#   dsn: "https://public_key/sentry.io/somethingsomething"
+
+# # Additional configuration for SAML authentication
+# # For more details, see https://github.com/handnot2/samly
+# config :samly, Samly.Provider,
+#   idp_id_from: :path_segment,
+#   service_providers: [
+#     %{
+#       id: "source-academy-backend",
+#       certfile: "example_path/certfile.pem",
+#       keyfile: "example_path/keyfile.pem"
+#     }
+#   ],
+#   identity_providers: [
+#     %{
+#       id: "student",
+#       sp_id: "source-academy-backend",
+#       base_url: "https://example_backend/sso",
+#       metadata_file: "student_idp_metadata.xml"
+#     },
+#     %{
+#       id: "staff",
+#       sp_id: "source-academy-backend",
+#       base_url: "https://example_backend/sso",
+#       metadata_file: "staff_idp_metadata.xml"
+#     }
+#   ]

config/test.exs

@@ -68,7 +68,13 @@ config :cadet,
            username: "E1234564"
            #  role: :student
          }
-       ]}
+       ]},
+    "saml" =>
+      {Cadet.Auth.Providers.SAML,
+       %{
+         assertion_extractor: Cadet.Auth.Providers.NusstfAssertionExtractor,
+         client_redirect_url: "https://cadet.frontend/login/callback"
+       }}
   },
   autograder: [
     lambda_name: "dummy"

lib/cadet/accounts/notification_type.ex

@@ -3,12 +3,12 @@ import EctoEnum
 defenum(Cadet.Accounts.NotificationType, :notification_type, [
   # Notifications for new assessments
   :new,
-  # Notifications for autograded assessments
-  :autograded,
-  # Notifications for manually graded assessments
-  :graded,
   # Notifications for unsubmitted submissions
   :unsubmitted,
   # Notifications for submitted assessments
-  :submitted
+  :submitted,
+  # Notifications for published grades
+  :published_grading,
+  # Notifications for unpublished grades
+  :unpublished_grading
 ])