missing test for backend/communities/organizations/models.py 🧪

[Abhi-Bohora]

Contributor checklist

• This pull request is on a separate branch and not the main branch
• I have run the tests for the backend and frontend depending on what's needed for my changes as described in the testing section of the contributing guide

---

Description

This PR covers the missing test for backend/communities/organizations/models.py 71 4 94% 71, 79, 88, 131

This test the string representation methods of several model classes in organizations/models.py

Related issue

• #1134

[netlify]

✅ Deploy Preview for activist-org canceled.

Name	Link
🔨 Latest commit	db88c96
🔍 Latest deploy log	https://app.netlify.com/sites/activist-org/deploys/67da94277279770008bafec1

[github-actions]

Thank you for the pull request! ❤️

The activist team will do our best to address your contribution as soon as we can. If you're not already a member of our public Matrix community, please consider joining! We'd suggest using Element as your Matrix client, and definitely join the General and Development rooms once you're in. Also consider attending our bi-weekly Saturday developer syncs! It'd be great to meet you 😊

[github-actions]

Maintainer Checklist

The following is a checklist for maintainers to make sure this process goes as well as possible. Feel free to address the points below yourself in further commits if you realize that actions are needed :)

• The TypeScript, pytest and formatting workflows within the PR checks do not indicate new errors in the files changed

• The Playwright end to end and Zap penetration tests have been ran and are passing (if necessary)

• The changelog has been updated with a description of the changes for the upcoming release and the corresponding issue (if necessary)

[andrewtavis]

No stress on the build errors, @Abhi-Bohora. There's something with how it's aligning with Netlify that's the problem. I'll look into this :)

[andrewtavis]

Frontend tests are going to fail on the above build, @Abhi-Bohora. The header rules are fixed, but for some reason we're getting a frontend error in node_modules right now 🤔 If you have any suggestions here, feel free to let us know! Obviously not a part of what you're doing here, but figured I'd ask :)

CC @mattburnett-repo who I'll also ask about thin in #1164

[andrewtavis]

CC @cquinn540 here as well :) This PR is simple, so maybe we can get the fix in here as it'll be in before @mattburnett-repo's in #1164. We had a dependency warning and I updated Babel for it. That dropped Netlify support until I updated to a patch version, but now we have a random frontend error in node_modules/@nuxtjs/i18n/dist/runtime/plugins/i18n.js:167:18 that we can see four times when running sudo yarn test in the frontend 🤔

[Abhi-Bohora]

Hmm, It seems like somehow it is redefining the locales and giving this error TypeError: Cannot redefine property: locales
And that error is generated from 2 different files while it is running

This error originated in "test/components/dropdown/DropdownTheme.spec.ts" test file. It doesn't mean the error was thrown inside the file itself, but while it was running.

and

This error originated in "test/pages/auth/sign-in.spec.ts" test file. It doesn't mean the error was thrown inside the file itself, but while it was running

[cquinn540]

@andrewtavis opened to PR to fix the frontend test typescript error: #1167

[andrewtavis]

Thanks @cquinn540! Just got back from a mini vacation and right back into work :) Really appreciate the help here! 😊

[andrewtavis]

Thanks for keeping at this, @Abhi-Bohora! Just a bit more to go before we close #1134! 😊