Skip to content
This repository has been archived by the owner on Sep 10, 2024. It is now read-only.

Add simple login test #93

Merged
merged 9 commits into from
Oct 20, 2016
Merged

Add simple login test #93

merged 9 commits into from
Oct 20, 2016

Conversation

rdeknijf
Copy link
Contributor

No description provided.

baskoopmans
baskoopmans reviewed Oct 11, 2016
View reviewed changes
example/tests/test_cms.py
from django.test import Client


class TestCMS:
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why use an old style class? https://stackoverflow.com/questions/54867/what-is-the-difference-between-old-style-and-new-style-classes-in-python

@rdeknijf
Copy link
Contributor Author

@mmarcos The failure is valid. This test exposes a problem in UserChangeForm. Django 1.8 does not support strip in Charfield.

@mmarcos mmarcos merged commit a28a0c9 into develop Oct 20, 2016
@mmarcos mmarcos deleted the feature/example_project_tests branch March 31, 2017 10:00
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@baskoopmans baskoopmans baskoopmans left review comments

@mmarcos mmarcos mmarcos approved these changes

Assignees

@mmarcos mmarcos

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@rdeknijf @mmarcos @baskoopmans