橘子
橘子
> Thank you @citrusreticulata for the nice and fast update! This is a huge improvement already. I have some more comments and suggestions, but I feel we are coming closer...
I don't know why but ALL integration tests failed on my local machine. Not just for iotdb output plugin:  But also for other output plugins,. For example, mongoDB: ...
I don't know why `testcontainers` didn't work. The [CI error log](https://app.circleci.com/pipelines/github/influxdata/telegraf/11930/workflows/01c03b41-d4d9-4282-b82b-851a84ccb9d1/jobs/194304) says: ``` 2022/08/02 05:51:17 Starting container id: e50fcba886ae image: testcontainers/ryuk:0.3.3 2022/08/02 05:51:18 Waiting for container id e50fcba886ae image: testcontainers/ryuk:0.3.3...
> For the testcontainers: The first time you use it, it will take some time as the system needs to download the docker container. I tested locally and reverting `27806bc7d179ca3604e19facbea52ff8a7fc9adb`,...
> For reference here my pimped and extended tests... [iotdb_test.go.txt](https://github.com/influxdata/telegraf/files/9240892/iotdb_test.go.txt) Thanks for your testcases! I'm working on this and I will add some of what you uploaded. Add I will...
> > Thanks for your testcases! I'm working on this and I will add some of what you uploaded. > > Add I will use anonymous function to make creating...
> > I use anonymous function to call newIoTDB() instead of creating an instance directly, which is shorter and easier to compare. Therefore, I didn't use variable like uintConversion, or...
So maybe dividing this group of tests into two functions is better for understand... I'd better do this.
I have solved the problem that the last test function is not easy to read. All CI tests passed. And according to actual functionality, I divided Metric Conversion tests into...
> > I designed it to do unit testing. Because IoTDB is likely to support other tags processing methods in the future, in this case modifyRecordsWithTags() will show different behaviors...