remove test_requirements

Author: jlwalke2

CONTRIBUTING.md

@@ -31,15 +31,13 @@ integration testing without requiring a running SAS Viya environment, the
 [Betamax](https://pypi.org/project/betamax/) package is used to record and
 replay network interactions.
 
-In addition, [Tox](https://tox.readthedocs.io) is used to test 
-compatibility with different Python version.
+In addition, [Tox](https://tox.readthedocs.io) is used to automating development tasks
+such as testing, linting, and building documentation.
 
 All packages required for development and testing are listed in
-[test_requirements.txt](test_requirements.txt) and can be easily installed with
-```
-pip install -r test_requirements.txt
-``` 
-
+[tox.ini](tox.ini), but it should be unecessary to install these manually.  Running `tox`
+from the project root directory will automatically build virtual environments for all
+Python interpreters found on the system and then install the required packages in those environments.
 
 Before a pull request will be accepted:
 - contributions must pass existing regression tests located in tests/

doc/index.rst

@@ -33,11 +33,7 @@ If not already present, these packages will be downloaded and install automatica
 The following additional packages are recommended for full functionality:
 
 - swat
-
-
-All required and recommended packages are listed in `requirements.txt` and can be installed easily with::
-
-    pip install -r requirements.txt
+- kerberos
 
 
 Installation
@@ -47,10 +43,14 @@ For basic functionality::
 
     pip install sasctl
 
-For full functionality::
 
+Functionality that depends on additional packages can be installed using the following::
+
+    pip install sasctl[swat]
+    pip install sasctl[kerberos]
     pip install sasctl[all]
 
+
 Quickstart
 ----------
 
@@ -370,11 +370,11 @@ the copyright to your contribution, this simply gives us permission to use and r
 part of the project.
 
 1. Fork the repository
-#. Ensure you're environment has the necessary packages:
+#. Run all unit and integration tests and ensure they pass.  This can be easily accomplished by running the following:
 
-  :command:`pip install -r test_requirements.txt`
+  :command:`tox`
 
-3. Run all unit and integration tests and ensure they pass.  If any tests fail, you should investigate and correct the failure *before* making any changes.
+3. If any tests fail, you should investigate and correct the failure *before* making any changes.
 #. Make your code changes
 #. Include new tests that validate your changes
 #. Rerun all unit and integration tests and ensure they pass.

src/sasctl/tasks.py

@@ -236,15 +236,20 @@ def publish_model(model, destination, code=None, max_retries=60,
     # Response for publish request includes link to check publish log
     job = mr._monitor_job(publish_req)
 
+    # Wait for status
+    # If job failed & overwrite -> check type of destination
+    # delete model from destination
+    # publish again
+
     if job.state.lower() == 'failed':
         pass
 
     log = request_link(job, 'publishingLog')
     msg = log.get('log').lstrip('SUCCESS===')
 
     # As of Viya 3.4 MAS converts module names to lower case.
-    # Since we can't rely on the request module name being preserved, try to parse the URL out of the response
-    # so we can retrieve the created module.
+    # Since we can't rely on the request module name being preserved, try to
+    # parse the URL out of the response so we can retrieve the created module.
     try:
         details = json.loads(msg)