Skip to content

Commit

Permalink
fix(MQTT): Increase stack size to 10k
Browse files Browse the repository at this point in the history
  • Loading branch information
becem-gharbi committed Sep 16, 2023
1 parent c7adacf commit e17716b
Show file tree
Hide file tree
Showing 8 changed files with 169 additions and 1 deletion.
2 changes: 1 addition & 1 deletion include/MQTT.h
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@
#include "Command.h"
#include "Report.h"

#define MQTT_TASK_STACK_SIZE 8 * 1024
#define MQTT_TASK_STACK_SIZE 10 * 1024
#define MQTT_KEEP_ALIVE_SEC 30
#define MQTT_NETWORK_TIMEOUT_MS 8000

Expand Down
5 changes: 5 additions & 0 deletions test/.gitignore
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
.pio
.vscode/.browse.c_cpp.db*
.vscode/c_cpp_properties.json
.vscode/launch.json
.vscode/ipch
10 changes: 10 additions & 0 deletions test/.vscode/extensions.json
Original file line number Diff line number Diff line change
@@ -0,0 +1,10 @@
{
// See http://go.microsoft.com/fwlink/?LinkId=827846
// for the documentation about the extensions.json format
"recommendations": [
"platformio.platformio-ide"
],
"unwantedRecommendations": [
"ms-vscode.cpptools-extension-pack"
]
}
39 changes: 39 additions & 0 deletions test/include/README
Original file line number Diff line number Diff line change
@@ -0,0 +1,39 @@

This directory is intended for project header files.

A header file is a file containing C declarations and macro definitions
to be shared between several project source files. You request the use of a
header file in your project source file (C, C++, etc) located in `src` folder
by including it, with the C preprocessing directive `#include'.

```src/main.c

#include "header.h"

int main (void)
{
...
}
```

Including a header file produces the same results as copying the header file
into each source file that needs it. Such copying would be time-consuming
and error-prone. With a header file, the related declarations appear
in only one place. If they need to be changed, they can be changed in one
place, and programs that include the header file will automatically use the
new version when next recompiled. The header file eliminates the labor of
finding and changing all the copies as well as the risk that a failure to
find one copy will result in inconsistencies within a program.

In C, the usual convention is to give header files names that end with `.h'.
It is most portable to use only letters, digits, dashes, and underscores in
header file names, and at most one dot.

Read more about using header files in official GCC documentation:

* Include Syntax
* Include Operation
* Once-Only Headers
* Computed Includes

https://gcc.gnu.org/onlinedocs/cpp/Header-Files.html
46 changes: 46 additions & 0 deletions test/lib/README
Original file line number Diff line number Diff line change
@@ -0,0 +1,46 @@

This directory is intended for project specific (private) libraries.
PlatformIO will compile them to static libraries and link into executable file.

The source code of each library should be placed in a an own separate directory
("lib/your_library_name/[here are source files]").

For example, see a structure of the following two libraries `Foo` and `Bar`:

|--lib
| |
| |--Bar
| | |--docs
| | |--examples
| | |--src
| | |- Bar.c
| | |- Bar.h
| | |- library.json (optional, custom build options, etc) https://docs.platformio.org/page/librarymanager/config.html
| |
| |--Foo
| | |- Foo.c
| | |- Foo.h
| |
| |- README --> THIS FILE
|
|- platformio.ini
|--src
|- main.c

and a contents of `src/main.c`:
```
#include <Foo.h>
#include <Bar.h>

int main (void)
{
...
}

```

PlatformIO Library Dependency Finder will find automatically dependent
libraries scanning project source files.

More information about PlatformIO Library Dependency Finder
- https://docs.platformio.org/page/librarymanager/ldf.html
17 changes: 17 additions & 0 deletions test/platformio.ini
Original file line number Diff line number Diff line change
@@ -0,0 +1,17 @@
; PlatformIO Project Configuration File
;
; Build options: build flags, source filter
; Upload options: custom upload port, speed and extra flags
; Library options: dependencies, extra library storages
; Advanced options: extra scripting
;
; Please visit documentation for the other options and examples
; https://docs.platformio.org/page/projectconf.html

[env:esp32dev]
platform = espressif32
board = esp32dev
framework = arduino
lib_deps = bg-dev/ESPAdmin@^0.1.4
monitor_raw = true
monitor_speed= 115200
40 changes: 40 additions & 0 deletions test/src/main.cpp
Original file line number Diff line number Diff line change
@@ -0,0 +1,40 @@
#include <Arduino.h>
#include <WiFi.h>
#include "ESPAdmin.h"

const char *httpHost = "esp-admin-app.bg.tn"; // Should be secured (over HTTPS)
const char *deviceId = "65043be911b195e46ae16a12"; // Device identifier
const char *apiKey = "tester123"; // Device API key
const char *ssid = "ASUS_AP"; // WiFi station SSID
const char *password = "pass1234"; // WiFi station password

ESPAdmin::Logger logger("app");

void setup()
{
Serial.begin(115200);
Serial.printf("\n");

WiFi.begin(ssid, password);

logger.info("WiFi wait");

while (WiFi.status() != WL_CONNECTED)
{
delay(1000);
}

logger.success("WiFi connected");

ESPAdmin::begin(httpHost, deviceId, apiKey);

pinMode(2, OUTPUT);
}

void loop()
{
digitalWrite(2, HIGH);
delay(500);
digitalWrite(2, LOW);
delay(1000);
}
11 changes: 11 additions & 0 deletions test/test/README
Original file line number Diff line number Diff line change
@@ -0,0 +1,11 @@

This directory is intended for PlatformIO Test Runner and project tests.

Unit Testing is a software testing method by which individual units of
source code, sets of one or more MCU program modules together with associated
control data, usage procedures, and operating procedures, are tested to
determine whether they are fit for use. Unit testing finds problems early
in the development cycle.

More information about PlatformIO Unit Testing:
- https://docs.platformio.org/en/latest/advanced/unit-testing/index.html

0 comments on commit e17716b

Please sign in to comment.