Fix a regression with NIS and improve code and testing.

Review Request #11050 — Created June 21, 2020 and submitted

Information

Review Board
release-3.0.x

Reviewers

The NIS backend regressed a while back when we reorganized the modules.
The import nis statements ended up importing our backend, rather than
Python's nis module. We only recently started hearing about this,
indicating that NIS authentication isn't widely used, but is still
used by some.

This change fixes the core issue by enabling
__future__.absolute_import, and enhances the code to be more testable
and address some old code quirks. A new unit test suite was added to
ensure that all NIS behavior works as expected.

Unit tests pass.

Summary ID
Fix a regression with NIS and improve code and testing.
The NIS backend regressed a while back when we reorganized the modules. The `import nis` statements ended up importing our backend, rather than Python's `nis` module. We only recently started hearing about this, indicating that NIS authentication isn't widely used, but *is* still used by some. This change fixes the core issue by enabling `__future__.absolute_import`, and enhances the code to be more testable and address some old code quirks. A new unit test suite was added to ensure that all NIS behavior works as expected.
6be6c80f3aec90775efe17fcaf4102e0ed7282a0
david
  1. Ship It!
  2. 
      
chipx86
Review request changed
Status:
Completed
Change Summary:
Pushed to release-3.0.x (9177675)