Quantcast
Channel: Issues for Drupal core
Viewing all articles
Browse latest Browse all 291708

Refactor Core Node Access Tests to not only test for denies, also test for public grants

$
0
0

Follow up for 1. in #1658846-187: Add language support to node access grants and records

Problem/Motivation

Missing test coverage for grants. We only have tests for denies.

Proposed resolution

Add more coverage for the case where there are actually explicit "public" grant records written from another module.

Remaining tasks

  • (optional) Check this issue summary for accuracy
  • (optional) Add some details to make this able to be picked up by someone new
  • Add the tests. See contributor task document for writing tests: http://drupal.org/node/1468170

User interface changes

No UI changes.

API changes

No API changes.

Original report by @xjm

We agreed earlier:
2. Add more coverage for the case where there are actually explicit "public" grant records written from another module or write a follow-up issue.

Fully agree, but as this is not yet covered currently I think we can make a follow up Issue: "Refactor Core Node Access Tests to not only test for denies, also test for public grants"

Let's file this and link it here.


Viewing all articles
Browse latest Browse all 291708

Trending Articles



<script src="https://jsc.adskeeper.com/r/s/rssing.com.1596347.js" async> </script>