[PR #5] [MERGED] Add basic test support via travis #59

Closed
opened 2026-02-26 03:33:36 +03:00 by kerem · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/laurivosandi/certidude/pull/5
Author: @plaes
Created: 9/14/2015
Status: Merged
Merged: 10/2/2015
Merged by: @laurivosandi

Base: masterHead: tests


📝 Commits (10+)

  • 4c1c201 Add basic tests
  • 81eef1d Remove the netifaces requirement
  • b87184b testing: Add Travis support
  • 5dc9ce4 Skip tests requiring certidude user
  • c68c5d2 Remove 'certidude' group requirement for creating CA configuration
  • 3e93aee cli: Make sure user doesn't accidentally overwrite existing setup
  • 706f4f7 cli: sha1 is deprecated, use sha256 instead.
  • 46b957b travis: Add python-3.5 target
  • 5d5894a Drop netifaces requirement
  • 5cddbe2 travis: Properly cache pip packages

📊 Changes

8 files changed (+125 additions, -35 deletions)

View changed files

📝 .gitignore (+1 -0)
.travis.yml (+16 -0)
📝 certidude/cli.py (+36 -23)
📝 certidude/helpers.py (+0 -3)
📝 certidude/wrappers.py (+22 -7)
📝 requirements.txt (+0 -1)
📝 setup.py (+0 -1)
tests/test_cli.py (+50 -0)

📄 Description

Tests require py.test and need to be documented. But there are currently more pressing things todo.. ;)


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/laurivosandi/certidude/pull/5 **Author:** [@plaes](https://github.com/plaes) **Created:** 9/14/2015 **Status:** ✅ Merged **Merged:** 10/2/2015 **Merged by:** [@laurivosandi](https://github.com/laurivosandi) **Base:** `master` ← **Head:** `tests` --- ### 📝 Commits (10+) - [`4c1c201`](https://github.com/laurivosandi/certidude/commit/4c1c2010c6197e0eff908c4e672711913e41320a) Add basic tests - [`81eef1d`](https://github.com/laurivosandi/certidude/commit/81eef1d42f9854bedf1b8872cf65f01be571ff25) Remove the netifaces requirement - [`b87184b`](https://github.com/laurivosandi/certidude/commit/b87184bcea055130eb1e68e6c2756013a126cee1) testing: Add Travis support - [`5dc9ce4`](https://github.com/laurivosandi/certidude/commit/5dc9ce46a252cdb8a9676c33fbfc1ae50da1d5bb) Skip tests requiring certidude user - [`c68c5d2`](https://github.com/laurivosandi/certidude/commit/c68c5d2a0706717bdeec38e01e15d234722117ca) Remove 'certidude' group requirement for creating CA configuration - [`3e93aee`](https://github.com/laurivosandi/certidude/commit/3e93aeee728f5eaae488f7991d465025e5b3bd2a) cli: Make sure user doesn't accidentally overwrite existing setup - [`706f4f7`](https://github.com/laurivosandi/certidude/commit/706f4f78d3eb3035892e6eefbee6de109e7ac98e) cli: sha1 is deprecated, use sha256 instead. - [`46b957b`](https://github.com/laurivosandi/certidude/commit/46b957b7578fe5f6844c5de6ba16aca8d4575814) travis: Add python-3.5 target - [`5d5894a`](https://github.com/laurivosandi/certidude/commit/5d5894a77a4b6ce01d42ffbeaf13f0ed809d636f) Drop netifaces requirement - [`5cddbe2`](https://github.com/laurivosandi/certidude/commit/5cddbe2a15879b85836b30f7b270e1fe762edc2b) travis: Properly cache pip packages ### 📊 Changes **8 files changed** (+125 additions, -35 deletions) <details> <summary>View changed files</summary> 📝 `.gitignore` (+1 -0) ➕ `.travis.yml` (+16 -0) 📝 `certidude/cli.py` (+36 -23) 📝 `certidude/helpers.py` (+0 -3) 📝 `certidude/wrappers.py` (+22 -7) 📝 `requirements.txt` (+0 -1) 📝 `setup.py` (+0 -1) ➕ `tests/test_cli.py` (+50 -0) </details> ### 📄 Description Tests require py.test and need to be documented. But there are currently more pressing things todo.. ;) --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
kerem 2026-02-26 03:33:36 +03:00
Sign in to join this conversation.
No labels
pull-request
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
starred/certidude-laurivosandi#59
No description provided.