Bring repo in line with edX standards

- Removed unused PEP 257 dependency
- Replaced Pylint with edx-lint
- Updated PEP 8 config file
- Using Make targets to run commands instead of shell script
- Using django-admin instead of copying manage.py file
- Removed unnecessary settings
- Removed unused mocks directory

ECOM-6862

.coveragerc

@@ -2,5 +2,4 @@
 branch = True
 source = organizations
 omit =
-    **/__init__.py
     **/migrations/*

.pep8

@@ -1,5 +1,4 @@
 [pep8]
 ignore=E501
 max_line_length=119
-exclude=settings
-exclude=migrations
+exclude=settings,migrations

.travis.yml

@@ -11,13 +11,12 @@ before_install:
 sudo: false
 
 install:
-    - "pip install -r requirements.txt"
+    - make requirements
     - "pip install coveralls"
 
 script:
-    - DJANGO_SETTINGS_MODULE=settings coverage run ./manage.py test organizations
-    - pep8 --config=.pep8 organizations
-    - pylint --rcfile=.pylintrc organizations --reports=no
+    - make test
+    - make quality
 
 after_success: coveralls
 

Makefile

@@ -0,0 +1,17 @@
+clean:
+	coverage erase
+	find . -name '*.pyc' -delete
+
+quality:
+	pep8 --config=.pep8 organizations
+	pylint --rcfile=pylintrc organizations
+
+requirements:
+	pip install -e .    # Install this package and its dependencies
+	pip install -r test-requirements.txt
+
+test:
+	django-admin.py test organizations --settings=test_settings --with-coverage --cover-package=organizations
+	coverage report
+
+.PHONY: clean, quality, requirements, test

README.rst

@@ -18,12 +18,14 @@ Usage
 Organizations is designed to centralize metadata about course publishers, such as their title, logo URL,
 and information included in certificates.
 
-Standalone Testing
-------------------
+Local Development
+-----------------
 
 .. code-block:: bash
 
-        $ ./run_tests
+    $ make requirements
+    $ make test
+    $ make quality
 
 Open edX Platform Integration
 -----------------------------

organizations/data.py

@@ -71,7 +71,7 @@ def _inactivate_organization(organization):
      in internal.Organization.objects.filter(id=organization.id, active=True)]
 
 
-def _activate_organization_course_relationship(relationship):
+def _activate_organization_course_relationship(relationship):  # pylint: disable=invalid-name
     """
     Activates an inactive organization-course relationship
     """
@@ -84,7 +84,7 @@ def _activate_organization_course_relationship(relationship):
     _activate_record(relationship)
 
 
-def _inactivate_organization_course_relationship(relationship):
+def _inactivate_organization_course_relationship(relationship):  # pylint: disable=invalid-name
     """
     Inactivates an active organization-course relationship
     """

organizations/models.py

@@ -1,6 +1,3 @@
-# pylint: disable=no-init
-# pylint: disable=old-style-class
-# pylint: disable=too-few-public-methods
 """
 Database ORM models managed by this Django app
 Please do not integrate directly with these models!!!  This app currently
@@ -43,7 +40,7 @@ class OrganizationCourse(TimeStampedModel):
     organization = models.ForeignKey(Organization, db_index=True)
     active = models.BooleanField(default=True)
 
-    class Meta:
+    class Meta(object):
         """ Meta class for this Django model """
         unique_together = (("course_id", "organization"),)
         verbose_name = _('Link Course')