Merge pull request #90 from bonitoo-io/fix/query-bool

fix: proper flux boolean data name

Author: vlastahajek

CHANGELOG.md

@@ -1,11 +1,14 @@
 # Changelog
 ## Version 3.4.0 (in progres)
 ### Features
- - [#88](https://github.com/tobiasschuerg/InfluxDB-Client-for-Arduino/pull/88) - ESP8266 only - Added Max Fragment Length Negotiation for TLS communicaton to reduce memory allocation. If server supports MFLN, it saves ~15kB. Standalone InfluxDB OSS server doesn't support MFLN, Cloud yes. To leverage MFLN for standalone OSS, a reverse proxy needs to be used. 
+ - [#89](https://github.com/tobiasschuerg/InfluxDB-Client-for-Arduino/pull/89) - ESP8266 only - Added Max Fragment Length Negotiation for TLS communicaton to reduce memory allocation. If server supports MFLN, it saves ~15kB. Standalone InfluxDB OSS server doesn't support MFLN, Cloud yes. To leverage MFLN for standalone OSS, a reverse proxy needs to be used. 
 
- ### Documentation
+### Documentation
  - [#87](https://github.com/tobiasschuerg/InfluxDB-Client-for-Arduino/pull/87) - Fixed include file name in the Readme
 
+### Fixes
+ - [#90](https://github.com/tobiasschuerg/InfluxDB-Client-for-Arduino/pull/90) - Fixed boolean type recognition of InfluxDB Flux
+
 ## Version 3.3.0 (2020-07-07)
  - [NEW] Added possibility skip server certification validation (`setInsecure()` method)
  - [NEW] Added possibility to query flux on secured InfuxDB 1.8 using V1 approach

src/query/FluxTypes.cpp

@@ -29,7 +29,7 @@
 
 const char	*FluxDatatypeString     = "string";
 const char	*FluxDatatypeDouble     = "double";
-const char	*FluxDatatypeBool       = "bool";
+const char	*FluxDatatypeBool       = "boolean";
 const char	*FluxDatatypeLong       = "long";
 const char	*FluxDatatypeUnsignedLong   = "unsignedLong";
 const char	*FluxDatatypeDuration       = "duration";

test/server/server.js

@@ -174,7 +174,7 @@ var queryRes = {
 ,_result1,1,2020-02-16T22:19:49.747562847Z,2020-02-17T22:19:49.747562847Z,2020-02-17T10:34:08.135814545Z,-4,i,test,1,adsfasdf
 ,_result1,1,2020-02-16T22:19:49.747562847Z,2020-02-17T22:19:49.747562847Z,2020-02-16T22:08:44.850214724Z,-1,i,test,1,adsfasdf
 \r
-#datatype,string,long,dateTime:RFC3339,dateTime:RFC3339,dateTime:RFC3339,bool,string,string,string,string
+#datatype,string,long,dateTime:RFC3339,dateTime:RFC3339,dateTime:RFC3339,boolean,string,string,string,string
 ,result,table,_start,_stop,_time,_value,_field,_measurement,a,b
 ,_result2,2,2020-02-17T22:19:49.747562847Z,2020-02-18T22:19:49.747562847Z,2020-02-18T10:34:08.135814545Z,false,b,test,0,brtfgh
 ,_result2,2,2020-02-17T22:19:49.747562847Z,2020-02-18T22:19:49.747562847Z,2020-02-18T22:08:44.969100374Z,true,b,test,0,brtfgh

test/test.ino

@@ -1316,12 +1316,12 @@ void testFluxParserMultiTables(bool chunked) {
     TEST_ASSERT(testStringValue(flux, 9, "b", "adsfasdf"));
 
 
-    TEST_ASSERTM(flux.next(),"flux.next()");
-    TEST_ASSERTM(flux.hasTableChanged(),"flux.hasTableChanged()");
+    TEST_ASSERTM(flux.next(),flux.getError());
+    TEST_ASSERTM(flux.hasTableChanged(),flux.getError());
     TEST_ASSERTM(flux.getError() == "",flux.getError());
 
      // ===== table 3 =================
-    const char *types3[] = {"string","long", "dateTime:RFC3339",  "dateTime:RFC3339",  "dateTime:RFC3339", "bool", "string","string","string","string"};
+    const char *types3[] = {"string","long", "dateTime:RFC3339",  "dateTime:RFC3339",  "dateTime:RFC3339", "boolean", "string","string","string","string"};
     TEST_ASSERT(testStringVector(flux.getColumnsDatatype(), types3, 10));
     TEST_ASSERT(testTableColumns(flux, columns, 10));
     // ========== row 1 ================