thevar1able/ClickHouse
1
0
Fork 0
mirror of https://github.com/ClickHouse/ClickHouse.git synced 2024-12-20 05:05:38 +00:00
Code Issues Packages Projects Releases Wiki Activity
a9a6e9f3b4
ClickHouse/tests/queries/0_stateless/02047_client_exception.expect

Ignoring revisions in .git-blame-ignore-revs. Click here to bypass and see the normal blame view.

41 lines
1.0 KiB
Plaintext
Raw Normal View History

Add test
2021-10-08 07:48:05 +00:00
#!/usr/bin/expect -f
Add exp_internal for expect tests This is to catch possible issues like 01565_reconnect_after_client_error in [1]. [1]: https://s3.amazonaws.com/clickhouse-test-reports/38417/e33f236d502fe9f7906392f4ceffaee44e3c4ce4/stateless_tests__thread__actions__[3/3].html Signed-off-by: Azat Khuzhin <a.khuzhin@semrush.com>
2022-07-02 09:05:52 +00:00
set basedir [file dirname $argv0]
set basename [file tail $argv0]
Better usability of "expect" tests: less trouble with running directly
2024-03-23 16:52:26 +00:00
if {[info exists env(CLICKHOUSE_TMP)]} {
set CLICKHOUSE_TMP $env(CLICKHOUSE_TMP)
} else {
set CLICKHOUSE_TMP "."
}
exp_internal -f $CLICKHOUSE_TMP/$basename.debuglog 0
set history_file $CLICKHOUSE_TMP/$basename.history
Add exp_internal for expect tests This is to catch possible issues like 01565_reconnect_after_client_error in [1]. [1]: https://s3.amazonaws.com/clickhouse-test-reports/38417/e33f236d502fe9f7906392f4ceffaee44e3c4ce4/stateless_tests__thread__actions__[3/3].html Signed-off-by: Azat Khuzhin <a.khuzhin@semrush.com>
2022-07-02 09:05:52 +00:00
Add test
2021-10-08 07:48:05 +00:00
log_user 0
Fix 02047_client_exception flakiness Signed-off-by: Azat Khuzhin <a.khuzhin@semrush.com>
2023-02-24 12:29:44 +00:00
set timeout 60
Add test
2021-10-08 07:48:05 +00:00
match_max 100000
expect_after {
Do not ignore eof in expect tests expect_after that adjusts default timeout handler, reseted eof handler, and this tell expect that it can ignore eof from read, consider the following example: <details> ```expect #!/usr/bin/expect -f exp_internal 1 log_user 1 set timeout 4 match_max 100000 expect_after { # eof { exp_continue } timeout { exit 1 } } spawn bash -c "sleep 1; echo ':) '" expect ":) " ``` ``` $ ./expect.expect < /dev/null spawn bash -c sleep 1; echo ':) ' parent: waiting for sync byte parent: telling child to go ahead parent: now unsynchronized from child spawn: returns {6614} expect: does "" (spawn_id exp4) match glob pattern ":) "? no expect: read eof ^^^ expect: set expect_out(spawn_id) "exp0" expect: set expect_out(buffer) "" ``` And with uncommented eof handler: ``` $ ./expect.expect < /dev/null spawn bash -c sleep 1; echo ':) ' parent: waiting for sync byte parent: telling child to go ahead parent: now unsynchronized from child spawn: returns {17959} expect: does "" (spawn_id exp4) match glob pattern ":) "? no expect: read eof expect: set expect_out(spawn_id) "exp0" expect: set expect_out(buffer) "" expect: continuing expect after update expect: does "" (spawn_id exp4) match glob pattern ":) "? no expect: does ":) \r\n" (spawn_id exp4) match glob pattern ":) "? yes expect: set expect_out(0,string) ":) " expect: set expect_out(spawn_id) "exp4" expect: set expect_out(buffer) ":) " ``` </details>
2021-12-28 18:16:04 +00:00
# Do not ignore eof from expect
Fix timeout for expect tests by fixing expect_after timeout By default expect matches patterns against output from the current process, however if you don't have stdin attached (like the case for CI), you will not have anything to check against, hence "timeout" simply does not work. Likely expect has $any_spawn_id (another option is to put expect_after after spawn). v2: use $any_spawn_id Refs: https://github.com/ClickHouse/ClickHouse/pull/43819 Signed-off-by: Azat Khuzhin <a.khuzhin@semrush.com>
2023-02-20 14:50:15 +00:00
-i $any_spawn_id eof { exp_continue }
Do not ignore eof in expect tests expect_after that adjusts default timeout handler, reseted eof handler, and this tell expect that it can ignore eof from read, consider the following example: <details> ```expect #!/usr/bin/expect -f exp_internal 1 log_user 1 set timeout 4 match_max 100000 expect_after { # eof { exp_continue } timeout { exit 1 } } spawn bash -c "sleep 1; echo ':) '" expect ":) " ``` ``` $ ./expect.expect < /dev/null spawn bash -c sleep 1; echo ':) ' parent: waiting for sync byte parent: telling child to go ahead parent: now unsynchronized from child spawn: returns {6614} expect: does "" (spawn_id exp4) match glob pattern ":) "? no expect: read eof ^^^ expect: set expect_out(spawn_id) "exp0" expect: set expect_out(buffer) "" ``` And with uncommented eof handler: ``` $ ./expect.expect < /dev/null spawn bash -c sleep 1; echo ':) ' parent: waiting for sync byte parent: telling child to go ahead parent: now unsynchronized from child spawn: returns {17959} expect: does "" (spawn_id exp4) match glob pattern ":) "? no expect: read eof expect: set expect_out(spawn_id) "exp0" expect: set expect_out(buffer) "" expect: continuing expect after update expect: does "" (spawn_id exp4) match glob pattern ":) "? no expect: does ":) \r\n" (spawn_id exp4) match glob pattern ":) "? yes expect: set expect_out(0,string) ":) " expect: set expect_out(spawn_id) "exp4" expect: set expect_out(buffer) ":) " ``` </details>
2021-12-28 18:16:04 +00:00
# A default timeout action is to do nothing, change it to fail
Fix timeout for expect tests by fixing expect_after timeout By default expect matches patterns against output from the current process, however if you don't have stdin attached (like the case for CI), you will not have anything to check against, hence "timeout" simply does not work. Likely expect has $any_spawn_id (another option is to put expect_after after spawn). v2: use $any_spawn_id Refs: https://github.com/ClickHouse/ClickHouse/pull/43819 Signed-off-by: Azat Khuzhin <a.khuzhin@semrush.com>
2023-02-20 14:50:15 +00:00
-i $any_spawn_id timeout { exit 1 }
Add test
2021-10-08 07:48:05 +00:00
}
adjust tests
2024-09-17 06:06:39 +00:00
spawn bash -c "source $basedir/../shell_config.sh ; \$CLICKHOUSE_CLIENT_BINARY \$CLICKHOUSE_CLIENT_OPT --disable_suggestion --enable-progress-table-toggle=0 --history_file=$history_file"
Add test
2021-10-08 07:48:05 +00:00
expect ":) "
send -- "DROP TABLE IF EXISTS test_02047\r"
expect "Ok."
send -- "CREATE TABLE test_02047 (s Int32) ENGINE=Memory()\r"
expect "Ok."
send -- "INSERT INTO test_02047 SELECT 'f' \r"
expect "Received exception from server"
send -- "DROP TABLE test_02047\r"
expect "Ok."
Fix 02047_client_exception flakiness Signed-off-by: Azat Khuzhin <a.khuzhin@semrush.com>
2023-02-24 12:29:44 +00:00
send -- "exit\r"
Add test
2021-10-08 07:48:05 +00:00
expect eof
Reference in New Issue Copy Permalink