{"thread":{"id":"53816","subject":"Git Test Coverage Report (July 7, 2020)","startedAt":"2020-07-08T00:02:21Z","lastAt":"2020-07-08T00:02:21Z","messageCount":1,"participants":["Derrick Stolee"],"isPatch":false,"patchVersion":null,"patchTotal":null},"messages":[{"id":"401168","messageId":"fa0df357-21da-46dd-7d73-a374b76448cb@gmail.com","threadId":"53816","inReplyTo":null,"subject":"Git Test Coverage Report (July 7, 2020)","fromName":"Derrick Stolee","fromEmail":"stolee@gmail.com","sentAt":"2020-07-08T00:02:16Z","receivedAt":"2020-07-08T00:02:21Z","isPatch":false,"sender":{"key":"stolee@gmail.com","avatar":"https://avatars.githubusercontent.com/u/570044?v=4"},"body":"Here is today's test coverage report.\n\nThanks,\n-Stolee\n\n[1] https://derrickstolee.github.io/git-test-coverage/reports/2020-07-07-commits.txt\n[2] https://derrickstolee.github.io/git-test-coverage/reports/2020-07-07.txt\n[3] https://derrickstolee.github.io/git-test-coverage/reports/2020-07-07.htm\n\n---\n\nseen\t66f3a24dd5403238838146bf7e526640ecb65ed7\njch\td9855b418b6ffe688a28097b01b040003bf6233f\nnext\tb3809f4674cea6e2f5db8a478a39b85851690c7d\nmaster\t4a0fcf9f760c9774be77f51e1e88a7499b53d2e2\nmaster@{1}\t101b3204f37606972b40fc17dec84560c22f69f6\n\n\nUncovered code in 'seen' not in 'jch'\n--------------------------------------------------------\n\nCommits introducing uncovered code:\nDamien Robert\t812a5889 remote.c: fix handling of %(push:remoteref)\nremote.c\n812a5889 1638) return NULL;\n812a5889 1645) return branch->refname;\n812a5889 1660) return NULL;\n812a5889 1774) return NULL;\n\nEmily Shaffer\t933f1b2e hook: add list command\nbuiltin/hook.c\n933f1b2e 42) printf(_(\"no commands configured for hook '%s'\\n\"),\n933f1b2e 44) return 0;\n\nhook.c\n933f1b2e 83) return NULL;\n\nJiang Xin\t1e58de8c receive-pack: add new proc-receive hook\nbuiltin/receive-pack.c\n1e58de8c 1046) return code;\n1e58de8c 1054) if (use_sideband)\n1e58de8c 1055) finish_async(&muxer);\n1e58de8c 1056) return code;\n1e58de8c 1542) cmd->report.error_message = \"broken symref\";\n1e58de8c 1727) cmd->report.error_message = \"transaction failed to start\";\n1e58de8c 1866) if (!cmd->report.error_message)\n1e58de8c 1867) cmd->report.error_message = \"unable to migrate objects to permanent storage\";\n\nt/helper/test-proc-receive.c\n1e58de8c 133) usage_msg_opt(\"Too many arguments.\", proc_receive_usage, options);\n\nJiang Xin\t5e4553bd New capability \"report-status-v2\" for git-push\nsend-pack.c\n5e4553bd 182)        if (new_options) {\n5e4553bd 184) new_options = 0;\n5e4553bd 186) ret = -1;\n5e4553bd 187) continue;\n5e4553bd 223) ret = -1;\n5e4553bd 224) break;\n5e4553bd 241)     hint->status != REF_STATUS_OK &&\n5e4553bd 242)     hint->status != REF_STATUS_REMOTE_REJECT) {\n5e4553bd 435) else if (server_supports(\"report-status\"))\n\ntransport-helper.c\n5e4553bd 837) hint = NULL;\n5e4553bd 856) (*ref)->report.message = msg;\n\nJiang Xin\t2dd1150a receive-pack: new config receive.procReceiveRefs\nbuiltin/receive-pack.c\n2dd1150a 239) return config_error_nonbool(var);\n2dd1150a 1839) continue;\n\nMatheus Tavares\ta2371e32 t/helper/test-config: facilitate addition of new cli options\nt/helper/test-config.c\na2371e32 103) goto print_usage_error;\na2371e32 160) printf(\"Value not found for \\\"%s\\\"\\n\", argv[1]);\na2371e32 167) printf(\"Value not found for \\\"%s\\\"\\n\", argv[1]);\na2371e32 175) printf(\"Value not found for \\\"%s\\\"\\n\", argv[1]);\na2371e32 205) printf(\"Value not found for \\\"%s\\\"\\n\", argv[1]);\na2371e32 232) printf(\"Value not found for \\\"%s\\\"\\n\", argv[1]);\n\nMatheus Tavares\t92bca22f config: correctly read worktree configs in submodules\nt/helper/test-config.c\n92bca22f 109) goto print_usage_error;\n92bca22f 114) fprintf(stderr, \"Cannot use --submodule with read_early_config\\n\");\n92bca22f 115) return TC_USAGE_ERROR;\n92bca22f 130) fprintf(stderr, \"Invalid argument to --submodule: '%s'\\n\",\n92bca22f 132) free(subrepo);\n92bca22f 133) ret = TC_USAGE_ERROR;\n92bca22f 134) goto out;\n92bca22f 187) fprintf(stderr, \"Cannot use --submodule with configset_get_value\\n\");\n92bca22f 188) ret = TC_USAGE_ERROR;\n92bca22f 189) goto out;\n92bca22f 210) fprintf(stderr, \"Cannot use --submodule with configset_get_value_multi\\n\");\n92bca22f 211) ret = TC_USAGE_ERROR;\n92bca22f 212) goto out;\n92bca22f 239) fprintf(stderr, \"Invalid syntax. Usage: test-tool config\"\n\nMatheus Tavares\tb77651be t/helper/test-config: return exit codes consistently\nt/helper/test-config.c\nb77651be 161) ret = TC_VALUE_NOT_FOUND;\nb77651be 168) ret = TC_VALUE_NOT_FOUND;\nb77651be 176) ret = TC_VALUE_NOT_FOUND;\nb77651be 206) ret = TC_VALUE_NOT_FOUND;\nb77651be 218) ret = TC_CONFIG_FILE_ERROR;\nb77651be 219) goto out;\nb77651be 233) ret = TC_VALUE_NOT_FOUND;\nb77651be 241) ret = TC_USAGE_ERROR;\n\nMatheus Tavares\te26e8bfc grep: honor sparse checkout patterns\nbuiltin/grep.c\ne26e8bfc 579) if (file_exists(sparse_file)) {\ne26e8bfc 583) free(sparse_file);\ne26e8bfc 584) free(patterns);\ne26e8bfc 585) return NULL;\n\nMiriam Rubio\t9aadf776 bisect--helper: introduce new `write_in_file()` function\nbuiltin/bisect--helper.c\n9aadf776 90) return error_errno(_(\"cannot open file '%s' in mode '%s'\"), path, mode);\n9aadf776 95) fclose(fp);\n9aadf776 96) return error_errno(_(\"could not write to file '%s'\"), path);\n\nPhillip Wood\t86696a9d rebase -i: support --ignore-date\nsequencer.c\n86696a9d 893) return NULL;\n86696a9d 1477) res = -1;\n86696a9d 1478) goto out;\n\nPhillip Wood\te12c40fd rebase -i: support --committer-date-is-author-date\nsequencer.c\ne12c40fd 1395) goto out;\ne12c40fd 1399) goto out;\n\nPranit Bauva\t10520dbf bisect--helper: reimplement `bisect_state` & `bisect_head` shell functions in C\nbuiltin/bisect--helper.c\n10520dbf 873) return BISECT_FAILED;\n10520dbf 889) oid_array_clear(&revs);\n10520dbf 890) return BISECT_FAILED;\n\nPranit Bauva\t05a69202 bisect--helper: reimplement `bisect_autostart` shell function in C\nbuiltin/bisect--helper.c\n05a69202 822) fprintf(stderr, _(\"You need to start by \\\"git bisect \"\n05a69202 825) if (!isatty(STDIN_FILENO))\n05a69202 826) return 1;\n05a69202 833) yesno = git_prompt(_(\"Do you want me to do it for you \"\n05a69202 835) if (starts_with(yesno, _(\"n\")) || starts_with(yesno, _(\"N\")))\n05a69202 836) return 1;\n05a69202 838) return bisect_start(terms, 0, NULL, 0);\n\nPranit Bauva\tf0bf9482 bisect--helper: finish porting `bisect_start()` to C\nbuiltin/bisect--helper.c\nf0bf9482 734) return BISECT_FAILED;\nf0bf9482 755) res = BISECT_FAILED;\nf0bf9482 767) res = BISECT_FAILED;\nf0bf9482 779) res = BISECT_FAILED;\n\nPranit Bauva\t587c9fac bisect--helper: reimplement `bisect_next` and `bisect_auto_next` shell functions in C\nbuiltin/bisect--helper.c\n587c9fac 492) return error_errno(_(\"failed to write to '%s'\"), git_path_bisect_log());\n587c9fac 521) return error_errno(_(\"could not open '%s' for appending\"),\n\nShawn O. Pearce\t38a81b4e receive-pack: Wrap status reports inside side-band-64k\nbuiltin/receive-pack.c\n38a81b4e 2283) send_sideband(1, 1, buf.buf, buf.len, use_sideband);\n\nTaylor Blau\t2129af73 list_objects_filter_options: introduce 'list_object_filter_config_name'\nlist-objects-filter-options.c\n2129af73 23) break;\n2129af73 36) break;\n\n\n\nUncovered code in 'jch' not in 'next'\n--------------------------------------------------------\n\nCommits introducing uncovered code:\nMichael Forney\tea3f7e59 revision: use repository from rev_info when parsing commits\nrevision.c\nea3f7e59 1062) if (repo_parse_commit(revs->repo, p) < 0)\n\n\n\nUncovered code in 'next' not in 'master'\n--------------------------------------------------------\n\nCommits introducing uncovered code:\nSZEDER Gábor\t17e6275f commit-graph: simplify chunk writes into loop\ncommit-graph.c\n17e6275f 1780) return -1;\n\n\n\nUncovered code in 'master' not in 'master@{1}'\n--------------------------------------------------------\n\nCommits introducing uncovered code:\nAbhishek Kumar\t4844812b commit-graph: introduce commit_graph_data_slab\ncommit-graph.c\n4844812b 1349) data->generation = GENERATION_NUMBER_MAX;\n\nbrian m. carlson\t122037c2 connect: add function to detect supported v1 hash functions\nconnect.c\n122037c2 566) hash = hash_algos[GIT_HASH_SHA1].name;\n122037c2 567) len = strlen(hash);\n122037c2 573) hash = next_server_feature_value(\"object-format\", &len, &offset);\n\nbrian m. carlson\t14570dc6 wrapper: add function to compare strings with different NUL termination\nwrapper.c\n14570dc6 112) return res;\n\nbrian m. carlson\t8b85ee4f transport-helper: implement object-format extensions\ntransport-helper.c\n8b85ee4f 1113) exit(128);\n\nbrian m. carlson\tac093d07 remote-curl: detect algorithm for dumb HTTP by size\nremote-curl.c\nac093d07 264) return NULL;\nac093d07 282)     transport_anonymize_url(url.buf));\n\nbrian m. carlson\t7f605017 remote-curl: implement object-format extensions\nremote-curl.c\n7f605017 199) algo = hash_algo_by_name(value);\n7f605017 200) if (algo == GIT_HASH_UNKNOWN)\n7f605017 202) options.hash_algo = &hash_algos[algo];\n\nbrian m. carlson\t4b831208 fetch-pack: parse and advertise the object-format capability\nfetch-pack.c\n4b831208 1212)     the_hash_algo->name, hash_name);\n4b831208 1214) } else if (hash_algo_by_ptr(the_hash_algo) != GIT_HASH_SHA1) {\n4b831208 1216)     the_hash_algo->name);\n\nbrian m. carlson\t9de0dd36 serve: advertise object-format capability for protocol v2\nconnect.c\n9de0dd36 477) reader->hash_algo = &hash_algos[GIT_HASH_SHA1];\n\nbrian m. carlson\t1349ffed connect: add function to fetch value of a v2 server capability\nconnect.c\n1349ffed 98) return 0;\n\nbrian m. carlson\t6161ce7b bundle: detect hash algorithm when reading refs\nbundle.c\n6161ce7b 33) return NULL;\n6161ce7b 70) status = -1;\n6161ce7b 71) break;\n\nbrian m. carlson\t88a09a55 builtin/show-index: provide options to determine hash algo\nbuiltin/show-index.c\n88a09a55 29) hash_algo = hash_algo_by_name(hash_name);\n88a09a55 30) if (hash_algo == GIT_HASH_UNKNOWN)\n88a09a55 32) repo_set_hash_algo(the_repository, hash_algo);\n\nbrian m. carlson\tbb095d08 builtin/receive-pack: detect when the server doesn't support our hash\nbuiltin/receive-pack.c\nbb095d08 1644) hash = hash_algos[GIT_HASH_SHA1].name;\nbb095d08 1645) len = strlen(hash);\n\nChris Torek\t8bfcb3a6 git diff: improve range handling\nbuiltin/diff.c\n8bfcb3a6 329) usage(builtin_diff_usage);\n\nChristian Couder\tf203a88c upload-pack: move keepalive to upload_pack_data\nupload-pack.c\nf203a88c 1193) data->keepalive = git_config_int(var, value);\nf203a88c 1194) if (!data->keepalive)\nf203a88c 1195) data->keepalive = -1;\n\nChristian Couder\tf8edd1ca upload-pack: move use_sideband to upload_pack_data\nupload-pack.c\nf8edd1ca 1039) data->use_sideband = DEFAULT_PACKET_MAX;\n\nChristian Couder\te9d882b8 upload-pack: change multi_ack to an enum\nupload-pack.c\ne9d882b8 1029) data->multi_ack = MULTI_ACK;\n\nElijah Newren\t051df3cf wt-status: show sparse checkout status as well\nwt-status.c\n051df3cf 1491) return;\n\nEric Sunshine\tc9b77f2c worktree: factor out repeated string literal\nbuiltin/worktree.c\nc9b77f2c 105) strbuf_addf(reason, _(\"unable to read gitdir file (%s)\"),\n\nEric Sunshine\t916133ef worktree: prune linked worktree referencing main worktree path\nbuiltin/worktree.c\n916133ef 166) return -1;\n\nJeff King\tb5a2068c upload-pack: actually use some upload_pack_data bitfields\nupload-pack.c\nb5a2068c 1076) data->no_progress = 1;\n\nJeff King\t750bb325 fast-export: store anonymized oids as hex strings\nbuiltin/fast-export.c\n750bb325 731)        anonymize_oid(oid_to_hex(&obj->oid)) :\n\nJeff King\t65b5d9fa fast-export: allow seeding the anonymized mapping\nbuiltin/fast-export.c\n65b5d9fa 177) ret = NULL;\n\nJeff King\ta0f65641 fast-export: stop storing lengths in anonymized hashmaps\nbuiltin/fast-export.c\na0f65641 149) b = container_of(entry_or_key, const struct anonymized_entry, hash);\na0f65641 150) return strcmp(a->orig, b->orig);\n\nJonathan Tan\tacaaca7d upload-pack: refactor reading of pack-objects out\nupload-pack.c\nacaaca7d 210) return readsz;\nacaaca7d 438) send_client_data(1, output_state.buffer, output_state.used,\n\nJonathan Tan\tdd4b732d upload-pack: send part of packfile response as uri\nupload-pack.c\ndd4b732d 1412) string_list_clear(&data->uri_protocols, 0);\n\nJonathan Tan\t8d5d2a34 http-fetch: support fetching packfiles by URL\nhttp-fetch.c\n8d5d2a34 128) commits = walker_targets_stdin(&commit_id, &write_ref);\n\nJonathan Tan\t8e6adb69 http-fetch: refactor into function\nhttp-fetch.c\n8e6adb69 30) walker_targets_free(commits, commit_id, write_ref);\n8e6adb69 33) fprintf(stderr,\n\nJonathan Tan\t9da69a65 fetch-pack: support more than one pack lockfile\nbuiltin/fetch-pack.c\n9da69a65 246) pack_lockfiles.items[i].string);\n\nPatrick Steinhardt\t67541597 refs: implement reference transaction hook\nrefs.c\n67541597 2047) return ret;\n67541597 2062) ret = -1;\n\nRandall S. Becker\tf64b6a1f bugreport.c: replace strbuf_write_fd with write_in_full\nbugreport.c\nf64b6a1f 184) die_errno(_(\"unable to write to %s\"), report_path.buf);\n\n\n"}]}