Skip to content
This repository has been archived by the owner on Jun 15, 2024. It is now read-only.

Gui integration test #95

Draft
wants to merge 50 commits into
base: master
Choose a base branch
from
Draft

Gui integration test #95

wants to merge 50 commits into from

Conversation

nh916
Copy link
Contributor

@nh916 nh916 commented May 27, 2023

Description

Testing the Excel Uploader GUI is a very manual process and if you need to test it multiple times it becomes a huge pain. This is a small script attempt to automate the process

Issue: #93

How To Run

  1. cd src/

  2. python gui_main.py

  3. Keep the terminal running

  4. in another terminal

    1. install pip install --upgrade selenium
    2. cd tests/GUI/test_gui.py
    3. python test_gui.py
  5. A Chrome window should open up

  6. Go to the Excel uploader page

  7. and then fill out the form and try to submit it

There is much more to work on and get perfect. It is still manual and has a bit more work for it to be fully automated, but this is something simple to show and get the imagination running

CVilla17 and others added 30 commits September 23, 2022 19:03
# Conflicts:
#	main.py
#	requirements.txt
removed requirements_dev.txt because it is no longer needed and just adds extra file
updated third party packages and tested the Excel uploader and created and executable and tested it again and it worked fine
removing docs CI/CD GitHub action, docs folder, mkdocs.yml file
* added pandas to dependencies and showing that we are using python 3.9+
* update requirements.txt 
* put requirements_dev.txt into requirements.txt
* removed requirements_dev.txt
updated workflow to use requirements.txt instead of requirements_dev.txt
* adding property:method
* Add characterization for exisitng materials functionality

* Tested add_material_characteristics and made fixes

* changed truth check

* Tested synthesis, works. Also, changed truth checking back

* made boolean assignment a function

* Variable for row input start
* version bump to 0.6.4

* version bump to 1.0.0
to be "CRIPT_Excel_Uploader_Mac_Zip" instead of before when it would come out to "CRIPT.Excel.Uploader.Mac.Zip"
updated requirements.txt to use CRIPT 0.6.1 instead of 0.6.0
updated Excel Uploader version to say 1.0.1 to reflect the patch
* Update issue templates

I think having good issue templates where users can report their issue with the needed information can benefit this repository and help our users

* Update README.md

* New Release (#60)

* bumping version to 0.6.3

* removed requirements_dev.txt

* updated packages on requirements.txt

* removing docs from master, so docs can only live in the docs branch

* Fix mac issues

* updated workflow (#54)

updated workflow to use requirements.txt instead of requirements_dev.txt

* updated importing tkinter (#55)

* Feature property method (#57)

* adding property:method

* Add material characteristics (#52)

* Add characterization for exisitng materials functionality

* Tested add_material_characteristics and made fixes

* changed truth check

* Tested synthesis, works. Also, changed truth checking back

* made boolean assignment a function

* Variable for row input start

* Version bump 1.0.0 (#59)

* version bump to 0.6.4

* version bump to 1.0.0

Co-authored-by: CVilla17 <villac@mit.edu>
Co-authored-by: Brilant Kasami <brili@users.noreply.github.com>
Co-authored-by: Bora Dauti <boradauti@Boras-MacBook-Pro.local>
Co-authored-by: CVilla17 <95596724+CVilla17@users.noreply.github.com>

Co-authored-by: Ardi <64595901+Ardi028@users.noreply.github.com>
Co-authored-by: Brilant Kasami <brili@users.noreply.github.com>
Co-authored-by: CVilla17 <villac@mit.edu>
Co-authored-by: Bora Dauti <boradauti@Boras-MacBook-Pro.local>
Co-authored-by: CVilla17 <95596724+CVilla17@users.noreply.github.com>
@nh916 nh916 requested a review from CVilla17 May 27, 2023 00:31
@nh916 nh916 marked this pull request as draft May 30, 2023 18:54
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants