arg: update test-arg-parser

Author: ServeurpersoCom

tests/test-arg-parser.cpp

@@ -37,6 +37,38 @@ int main(void) {
                         exit(1);
                     }
                 }
+
+                // ensure short argument precedes long argument
+                if (opt.args.size() > 1) {
+                    const std::string first(opt.args.front());
+                    const std::string last(opt.args.back());
+
+                    // only check if we have a mix of short (-x) and long (--xxx) forms
+                    bool first_is_short = (first.size() >= 2 && first[0] == '-' && first[1] != '-');
+                    bool last_is_short = (last.size() >= 2 && last[0] == '-' && last[1] != '-');
+
+                    // if at least one is short, verify order
+                    if ((first_is_short || last_is_short) && first.length() > last.length()) {
+                        fprintf(stderr, "test-arg-parser: short form should come before long form: %s, %s\n",
+                                first.c_str(), last.c_str());
+                        assert(false);
+                    }
+                }
+
+                // same check for negated arguments
+                if (opt.args_neg.size() > 1) {
+                    const std::string first(opt.args_neg.front());
+                    const std::string last(opt.args_neg.back());
+
+                    bool first_is_short = (first.size() >= 2 && first[0] == '-' && first[1] != '-');
+                    bool last_is_short = (last.size() >= 2 && last[0] == '-' && last[1] != '-');
+
+                    if ((first_is_short || last_is_short) && first.length() > last.length()) {
+                        fprintf(stderr, "test-arg-parser: short negated form should come before long form: %s, %s\n",
+                                first.c_str(), last.c_str());
+                        assert(false);
+                    }
+                }
             }
         } catch (std::exception & e) {
             printf("%s\n", e.what());