Skip to content

m03 test adequacy - #9

Merged
iamabhutta merged 18 commits into
mainfrom
m03-test-adequacy
Sep 12, 2026
Merged

iamabhutta merged 18 commits into
mainfrom
m03-test-adequacy

Conversation

@iamabhutta

Copy link
Copy Markdown
Contributor

Starting with Sum26 slides
TODO:
-Update as needed
-Review activities and update as needed

@netlify

netlify Bot commented Aug 8, 2026 •

Copy link
Copy Markdown

✅ Deploy Preview for cs4530fall26 ready!

Name Link
🔨 Latest commit 48a78d4
🔍 Latest deploy log https://app.netlify.com/projects/cs4530fall26/deploys/6aa4adcac330ca0008c73bc1
😎 Deploy Preview https://deploy-preview-9--cs4530fall26.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@iamabhutta
iamabhutta requested a review from mwand August 8, 2026 11:53
@mwand

mwand commented Aug 19, 2026

Copy link
Copy Markdown
Collaborator

Now just a single slide deck and a single activity. Rewrote Learning Objective to match new slide deck. I still haven't gotten l3.md and modules.yml to work together. Can you fix this?

@iamabhutta

Copy link
Copy Markdown
Contributor Author

fixed. It was missing a dash.

@iamabhutta

Copy link
Copy Markdown
Contributor Author

what's the need for this? I thought link was working fine.

@mwand

mwand commented Aug 19, 2026

Copy link
Copy Markdown
Collaborator

Still need to check that activity runs as advertised. Maybe have TA check. Otherwise LGTM.

Note that we no longer use Postman. I'm guessing that most of our uses of Postman can be replaced by supertest. (Have I mentioned lately that I hate Postman?). There are several good tutorial intros for Supertest; no need to write our own tutorial (yay!).

@iamabhutta

Copy link
Copy Markdown
Contributor Author

Manas checked the code for activity and confirmed that it works.
We also removed the postman.
Ready to merge, when we need (probably Friday 9/12)

@iamabhutta
iamabhutta merged commit d12f7ff into main Sep 12, 2026
4 checks passed
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.

2 participants