starred/WiFiManager
1
0
Fork 0
mirror of https://github.com/tzapu/WiFiManager.git synced 2026-04-27 00:55:52 +03:00
Code Issues 656 Projects Releases 8 Packages Wiki Activity

[GH-ISSUE #969] Continues Integration: automatic hardware tests #823

New issue
Open
opened 2026-02-28 01:27:13 +03:00 by kerem · 11 comments
kerem commented 2026-02-28 01:27:13 +03:00
Owner
Copy link

Originally created by @dontsovcmc on GitHub (Nov 15, 2019).
Original GitHub issue: https://github.com/tzapu/WiFiManager/issues/969

It's comfortable way to refactor library is to have a lot of tests.
I can write small software tests, but hardware tests are more interesting.

Idea:

  1. use platformio for command line build WiFiManager test examples
  2. connect to ESP8266 using python or bash language. For example:
    https://github.com/rockymeza/wifi
  3. test web pages content by HTTP requests
  4. check items, push buttons, fill labels by SeleniumHQ and etc

Pros:

  1. We can do a lot of refactor safety.

Cons:

  1. Control PC Wi-Fi is platform dependency.
Originally created by @dontsovcmc on GitHub (Nov 15, 2019). Original GitHub issue: https://github.com/tzapu/WiFiManager/issues/969 It's comfortable way to refactor library is to have a lot of tests. I can write small software tests, but hardware tests are more interesting. Idea: 1. use platformio for command line build WiFiManager test examples 2. connect to ESP8266 using python or bash language. For example: https://github.com/rockymeza/wifi 3. test web pages content by HTTP requests 4. check items, push buttons, fill labels by SeleniumHQ and etc Pros: 1. We can do a lot of refactor safety. Cons: 1. Control PC Wi-Fi is platform dependency.
kerem commented 2026-02-28 01:27:13 +03:00
Author
Owner
Copy link

@dontsovcmc commented on GitHub (Dec 2, 2019):

@tablatronix I made it =)
Now we can check pages and refactor code safety:
https://github.com/dontsovcmc/ESPWebServerTest

What is you OS? Now I support MacOS Wi-fi adapter.

<!-- gh-comment-id:560192497 --> @dontsovcmc commented on GitHub (Dec 2, 2019): @tablatronix I made it =) Now we can check pages and refactor code safety: https://github.com/dontsovcmc/ESPWebServerTest What is you OS? Now I support MacOS Wi-fi adapter.
kerem commented 2026-02-28 01:27:13 +03:00
Author
Owner
Copy link

@dontsovcmc commented on GitHub (Dec 5, 2019):

Look at examples/DEV/OnDemandConfigPortal test:
https://github.com/dontsovcmc/ESPWebServerTest#wifimanager-example-test

here is the Test:
https://github.com/dontsovcmc/ESPWebServerTest/blob/master/examples/WifiManagerTest/OnDemandConfigPortal/test_configportal.py

note: please, move handleRoute(), getTime() functions upper, cause compile error

<!-- gh-comment-id:561938454 --> @dontsovcmc commented on GitHub (Dec 5, 2019): Look at` examples/DEV/OnDemandConfigPortal` test: https://github.com/dontsovcmc/ESPWebServerTest#wifimanager-example-test here is the Test: https://github.com/dontsovcmc/ESPWebServerTest/blob/master/examples/WifiManagerTest/OnDemandConfigPortal/test_configportal.py note: please, move handleRoute(), getTime() functions upper, cause compile error
kerem commented 2026-02-28 01:27:13 +03:00
Author
Owner
Copy link

@tablatronix commented on GitHub (Dec 5, 2019):

What compile error?

<!-- gh-comment-id:561957657 --> @tablatronix commented on GitHub (Dec 5, 2019): What compile error?
kerem commented 2026-02-28 01:27:13 +03:00
Author
Owner
Copy link

@dontsovcmc commented on GitHub (Dec 5, 2019):

In C++ you should define function before use (PlatformIO works as C++).
Arduino IDE hacks this rule. So, add

void handleRoute();
void getTime();

before functions or mode them body upper.

<!-- gh-comment-id:562085856 --> @dontsovcmc commented on GitHub (Dec 5, 2019): In C++ you should define function before use (PlatformIO works as C++). Arduino IDE hacks this rule. So, add ``` void handleRoute(); void getTime(); ``` before functions or mode them body upper.
kerem commented 2026-02-28 01:27:14 +03:00
Author
Owner
Copy link

@tablatronix commented on GitHub (Dec 5, 2019):

where?

<!-- gh-comment-id:562102860 --> @tablatronix commented on GitHub (Dec 5, 2019): where?
kerem commented 2026-02-28 01:27:14 +03:00
Author
Owner
Copy link

@dontsovcmc commented on GitHub (Dec 5, 2019):

https://github.com/tzapu/WiFiManager/blob/development/examples/DEV/OnDemandConfigPortal/OnDemandConfigPortal.ino

<!-- gh-comment-id:562111497 --> @dontsovcmc commented on GitHub (Dec 5, 2019): https://github.com/tzapu/WiFiManager/blob/development/examples/DEV/OnDemandConfigPortal/OnDemandConfigPortal.ino
kerem commented 2026-02-28 01:27:14 +03:00
Author
Owner
Copy link

@tablatronix commented on GitHub (Dec 5, 2019):

I use platformio, I get no compile errors.

<!-- gh-comment-id:562182714 --> @tablatronix commented on GitHub (Dec 5, 2019): I use platformio, I get no compile errors.
kerem commented 2026-02-28 01:27:14 +03:00
Author
Owner
Copy link

@tablatronix commented on GitHub (Dec 5, 2019):

Are you compiling a copy or stream as cpp not ino ?

<!-- gh-comment-id:562188843 --> @tablatronix commented on GitHub (Dec 5, 2019): Are you compiling a copy or stream as cpp not ino ?
kerem commented 2026-02-28 01:27:14 +03:00
Author
Owner
Copy link

@dontsovcmc commented on GitHub (Dec 5, 2019):

Rename ino to cpp and compile... Platformio can eat ino?!

<!-- gh-comment-id:562342338 --> @dontsovcmc commented on GitHub (Dec 5, 2019): Rename ino to cpp and compile... Platformio can eat ino?!
kerem commented 2026-02-28 01:27:14 +03:00
Author
Owner
Copy link

@tablatronix commented on GitHub (Dec 5, 2019):

Yes

<!-- gh-comment-id:562354129 --> @tablatronix commented on GitHub (Dec 5, 2019): Yes
kerem commented 2026-02-28 01:27:14 +03:00
Author
Owner
Copy link

@dontsovcmc commented on GitHub (Dec 5, 2019):

Nice, simplify script.

<!-- gh-comment-id:562358715 --> @dontsovcmc commented on GitHub (Dec 5, 2019): Nice, simplify script.
Sign in to join this conversation.
No Branch/Tag specified
Branches Tags
master
revert-1849-master
feature_fastconnect
feature_auth
feature_slimable
PEAP
feature_statichtmlparams
feature_asyncwebserver
feature_pagestringsref
parameter-smart-ptr
gh-pages
Legacy_v1
hotfixes
autochannelselect_feature
travis
smartconfig
v2.0.17
v2.0.16-rc.2
v2.0.15-rc.1
v2.0.0-rc.1
v2.0.14-beta
v2.0.13-beta
v2.0.12-beta
v2.0.11-beta
v2.0.10-beta
v2.0.9-beta
v2.0.8-beta
2.0.7-beta
2.0.6-beta
2.0.5-beta
2.0.4-beta
0.16.0
2.0.3-alpha
0.15.0
0.14
0.13
0.12
0.11
0.10
0.9
0.8
0.7
0.6
0.5
v0.4
v0.3
v0.2
v0.1
Labels
Clear labels   
📶 WiFi

   
🕸️ HTTP

   
Branch

   
DEV Help Wanted

   
Discussion

   
Documentation

   
ESP32

   
Example

   
Good First Issue

   
Hotfix

   
In Progress

   
Incomplete

   
Needs Feeback

   
Priority

   
QA

   
Question

   
Task

   
Upstream/Dependancy

   
bug

   
duplicate

   
enhancement

   
invalid

   
pull-request

Mirrored from GitHub Pull Request

  
wontfix
No labels
📶 WiFi
🕸️ HTTP
Branch
DEV Help Wanted
Discussion
Documentation
ESP32
Example
Good First Issue
Hotfix
In Progress
Incomplete
Needs Feeback
Priority
QA
Question
Task
Upstream/Dependancy
bug
duplicate
enhancement
invalid
pull-request
wontfix
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
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/WiFiManager#823
No description provided.