Problem/Motivation

Although this module is in a (relatively) stable state now, it has very few tests.

We should write some more, particularly around some of the form interactions and permission checking.

Issue fork mastodon_api-3585658

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

philipnorton42 created an issue. See original summary.

philipnorton42’s picture

philipnorton42 commented

Add tests for access to the different forms based on user permissions.
- Add `administer mastodon api` and check access to the admin area.
- Add `administer mastodon_api_instance` and check access to the instance form.

Maybe write better documentation for the permissions.

stevenpatz’s picture

stevenpatz
he/him
commented

Add `administer mastodon api` and check access to the admin area.

Was looking at this issue today, isn't there already a test for this?

philipnorton42’s picture

philipnorton42 commented
Status: Active » Fixed

Hi stevenpatz,

Yes. You are right. We do have a test for that, in the MastodonApiPushStatusFormTest class.

I've been adding tests to the module when writing other bits of functionality. As a result, I think we are ok in terms of test coverage at the moment.

I've also been adding lots to the documentation over the last few months. I think the documentation is ok now.

What I'll do is close this issue. If you do have some code that you want to submit then please re-open the issue and add your code.

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.