Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Transfer all tests to a tests module #48

Merged
merged 15 commits into from
Nov 25, 2022
Merged

Transfer all tests to a tests module #48

merged 15 commits into from
Nov 25, 2022

Conversation

wanjirumurira
Copy link
Contributor

Fixes: #43

victorazangu and others added 15 commits November 1, 2022 18:05
@victorazangu
blog post model
70fdaeb
@victorazangu
merging from main branch
78969d9
@JimmyTron
Merge pull request #20 from victorazangu/victor-blog-post
4c0348d 
Victor blog post
@victorazangu
was inplementing post apis
3825214
@victorazangu
fixing merge conflict
d618cfb
@victorazangu
fixing merge conflict
8b9f928
@JimmyTron
removed unused imports from the models files
71b563f
@JimmyTron
minor tweak to the views retun msg
26c18d9
@victorazangu
created test folder
7ca44ff
@JimmyTron
Merge branch 'Dev' into victor-blog-post
e1505ca
@JimmyTron
Merge pull request #28 from victorazangu/victor-blog-post
12fca9a 
Victor blog post
@Eva-Wamuyu @hellen-22 @sangkips
Add unittests for user model (#41)
d09e203 
* Substituting the User Model (#34)

* creating accounts app for the user model

* creating user model

* creating accounts model

* registering account model to the admin

* commenting code

* changing the account model from the blog app to account app

* model registering

* add blog app to the installed apps in the settings file

* adding max_length to email field

* add user testclass from testcase

Co-authored-by: Hellen Wainaina <58620060+hellen-22@users.noreply.github.com>
Co-authored-by: Hellen Wainaina <hellenwain@gmail.com>
Co-authored-by: Kipkoech Sang <sangkipkoech@gmail.com>
@Eva-Wamuyu @hellen-22 @sangkips
add test folder in the Project i.e CMS folder and delete test.py file…
0bff2bb 
…s in the apps (#44)

* Substituting the User Model (#34)

* creating accounts app for the user model

* creating user model

* creating accounts model

* registering account model to the admin

* commenting code

* changing the account model from the blog app to account app

* model registering

* add blog app to the installed apps in the settings file

* adding max_length to email field

* add user testclass from testcase

* add test folder in project i.e CMS and modify installed files list

* rename folders to ^test

Co-authored-by: Hellen Wainaina <58620060+hellen-22@users.noreply.github.com>
Co-authored-by: Hellen Wainaina <hellenwain@gmail.com>
Co-authored-by: Kipkoech Sang <sangkipkoech@gmail.com>
@wanjirumurira
creating separate test folder
de5f5d6
@wanjirumurira
creating separate test folder
80af46d
nyandika
nyandika approved these changes Nov 23, 2022
View reviewed changes
CMS/settings.py Show resolved Hide resolved
@JimmyTron JimmyTron merged commit 332dd7e into main Nov 25, 2022
JimmyTron added a commit that referenced this pull request Nov 25, 2022
@JimmyTron
Revert "Transfer all tests to a tests module (#48)"
d7be088 
This reverts commit 332dd7e.
@JimmyTron JimmyTron mentioned this pull request Nov 25, 2022
Merged
JimmyTron added a commit that referenced this pull request Nov 25, 2022
@JimmyTron
Revert "Transfer all tests to a tests module (#48)" (#50)
981e63a 
This reverts commit 332dd7e.
@JimmyTron
Copy link
Member

Screenshot_2022-11-25-14-50-07-026_com android chrome

you could have a pages/tests/ directory and within it decided files--all starting with test_--for different areas of the app.

In this instance we have ../tests/models.py instead we expected ../tests/test_models.py

@JimmyTron JimmyTron mentioned this pull request Dec 2, 2022
Closed
JimmyTron added a commit that referenced this pull request Dec 19, 2022
@teslima0 @hellen-22
@sangkips @jumalaw98 @aibunny @JimmyTron @Eugene-Kwaka @wanjirumurira @victorazangu @Eva-Wamuyu @AmbroseOtundo @tallninja
User-autentication (#59)
2896873 
* Substituting the User Model (#34)

* creating accounts app for the user model

* creating user model

* creating accounts model

* registering account model to the admin

* commenting code

* changing the account model from the blog app to account app

* model registering

* add blog app to the installed apps in the settings file

* adding max_length to email field

* Update README.md

Read me update

* Unit tests of Account and User models (#40)

Co-authored-by: madbunny <madbunny@madbunny.madbunny>

* update README (#42)

* Update README.md

* Update README.md

* changes on the admin page for users

* update on the comments on the admin

* Create contributions.md

* Update phone number field to use international format

* Update README.md file to have PhoneNumber field settings

* updated phone number field to allow localizatuion option

* Add Phone number module

modified:   README.md
modified:   requirements.txt

* Transfer all tests to a tests module  (#48)

* blog post model

* was inplementing post apis

* fixing merge conflict

* removed unused imports from the models files

* minor tweak to the views retun msg

* created test folder

* Add unittests for  user model (#41)

* Substituting the User Model (#34)

* creating accounts app for the user model

* creating user model

* creating accounts model

* registering account model to the admin

* commenting code

* changing the account model from the blog app to account app

* model registering

* add blog app to the installed apps in the settings file

* adding max_length to email field

* add user testclass from testcase

Co-authored-by: Hellen Wainaina <58620060+hellen-22@users.noreply.github.com>
Co-authored-by: Hellen Wainaina <hellenwain@gmail.com>
Co-authored-by: Kipkoech Sang <sangkipkoech@gmail.com>

* add test folder in the Project i.e CMS folder and delete test.py files in the apps (#44)

* Substituting the User Model (#34)

* creating accounts app for the user model

* creating user model

* creating accounts model

* registering account model to the admin

* commenting code

* changing the account model from the blog app to account app

* model registering

* add blog app to the installed apps in the settings file

* adding max_length to email field

* add user testclass from testcase

* add test folder in project i.e CMS and modify installed files list

* rename folders to ^test

Co-authored-by: Hellen Wainaina <58620060+hellen-22@users.noreply.github.com>
Co-authored-by: Hellen Wainaina <hellenwain@gmail.com>
Co-authored-by: Kipkoech Sang <sangkipkoech@gmail.com>

Co-authored-by: victorazangu <victorsazangu@gmail.com>
Co-authored-by: Jimmy Oty <jimmytron17@gmail.com>
Co-authored-by: Jimmy Tron <oduorjames@hotmail.com>
Co-authored-by: Wamúyú <rutheve.eva@gmail.com>
Co-authored-by: Hellen Wainaina <58620060+hellen-22@users.noreply.github.com>
Co-authored-by: Hellen Wainaina <hellenwain@gmail.com>
Co-authored-by: Kipkoech Sang <sangkipkoech@gmail.com>

* Revert "Transfer all tests to a tests module  (#48)" (#50)

This reverts commit 332dd7e.

* Update issue templates

* Create django.yml

* github templates  (#52)

* Update README.md

* Update README.md

* Fix phone number bug

additon of the phone number field to the admin.py file. making the field
both visible on the update user and add user pages.

Co-authered-by  Hellen <https://github.com/hellen-22>
	modified:   accounts/admin.py

* Create django.yml

* Update README.md

* Creation of a pull request template

this template is to help guide us Write descriptive pull requests

	new file:   pull_request_template.md

* Update README.md

* Update django.yml

* Update django.yml

* Transfer tests (#53)

* doc:update README

* add tests module in accounts and blog folders and delete all tests.py files

* handling Images (#54)

* feat:default phone number to have  +254 isocode when creating superuser (#56)

* Update README.md

* Tamar-autentication

* Update README.md

* Wanjirumurira (#60)

* handling Images

* adding a default display picture

* Changing the site titles (#58)

admin.site.site_header = "SpaceYaTech Admin"
admin.site.site_title = "SpaceYaTech Admin Portal"
admin.site.index_title = "Welcome to SpaceYaTech Techies Portal"

	modified:   CMS/urls.py

* setting

* resolve

* urls-model

* update

* admin-urls-update

* Clear unwanted files from the VC (git)
Removing the batch file and also the images that were uploaded
this is so that they dont occupy space

	modified:   .gitignore
	deleted:    local.bat
	deleted:    media/blank-profile-picture.png
	deleted:    tamar.png

* Update CMS/urls.py

Co-authored-by: Ernest Wambua <43074705+tallninja@users.noreply.github.com>

Co-authored-by: Hellen Wainaina <58620060+hellen-22@users.noreply.github.com>
Co-authored-by: Hellen Wainaina <hellenwain@gmail.com>
Co-authored-by: Kipkoech Sang <sangkipkoech@gmail.com>
Co-authored-by: Lawrence Juma <41073697+jumalaw98@users.noreply.github.com>
Co-authored-by: Fredrick Waihenya <98146814+Fredrick-Waihenya@users.noreply.github.com>
Co-authored-by: madbunny <madbunny@madbunny.madbunny>
Co-authored-by: Jimmy Oty <jimmytron17@gmail.com>
Co-authored-by: Eugene Kwaka <kwakaeugene@gmail.com>
Co-authored-by: Jimmy Tron <oduorjames@hotmail.com>
Co-authored-by: Wanjiru Murira <wanjirumurira@gmail.com>
Co-authored-by: victorazangu <victorsazangu@gmail.com>
Co-authored-by: Wamúyú <rutheve.eva@gmail.com>
Co-authored-by: Ambrose Otundo <48138192+AmbroseOtundo@users.noreply.github.com>
Co-authored-by: Ernest Wambua <43074705+tallninja@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sangkips sangkips sangkips left review comments

@JimmyTron JimmyTron JimmyTron approved these changes

@nyandika nyandika nyandika approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Transfer all tests to a tests module (REOPENED)
6 participants
@wanjirumurira @JimmyTron @nyandika @sangkips @victorazangu @Eva-Wamuyu