bregman-arie / bregman-arie/devops-exercises

Test suite

Open
#51 7 comments 2 reactions 1 assignee Claimed by @surister View on GitHub
enhancement wip
Dominant language
Python
Stars
84.4k
Forks
20.3k
PR merge metrics
No merged PRs in 30d

Description

Issue merges from #50 .

While trying to do a simple bash script of this I came across many problems since there are many ways of spacing/formatting within the same layout, the bash script was getting a bit bloated and decided to make it in python.

One of the issues of counting questions and answers are typos. So I wrote a test suit to check for these, it's working great.

![imagen](https://user-images.githubusercontent.com/37985796/70993308-05be5b80-20cc-11ea-84fd-4996fadd7e06.png)

Notice the missing '/' from the closing summary tag.

So my new proposal is:

+ Add #50 (Already aproved)
+ Create test suite where we will be checking:
1. typos in tags
2. layout correctly followed
3. Correct number of answered questions count
4. Correct number of question count (The bash script that counts questions may not be counting correctly due to typos in tags)

Having this test suit will allow us to check our commits before pushing thus improving the general quality.

Also we could land a CI pipe for new pull requests, offering faster feedback to contributors.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.