<?xml version="1.0" encoding="UTF-8" standalone="yes" ?>
<!DOCTYPE bugzilla SYSTEM "https://bugs.webkit.org/page.cgi?id=bugzilla.dtd">

<bugzilla version="5.0.4.1"
          urlbase="https://bugs.webkit.org/"
          
          maintainer="admin@webkit.org"
>

    <bug>
          <bug_id>197298</bug_id>
          
          <creation_ts>2019-04-25 15:23:35 -0700</creation_ts>
          <short_desc>webkitpy: Including skipped tests in the final count</short_desc>
          <delta_ts>2019-04-25 17:17:46 -0700</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>1</classification_id>
          <classification>Unclassified</classification>
          <product>WebKit</product>
          <component>Tools / Tests</component>
          <version>WebKit Nightly Build</version>
          <rep_platform>Unspecified</rep_platform>
          <op_sys>Unspecified</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords>InRadar</keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Jonathan Bedard">jbedard</reporter>
          <assigned_to name="Jonathan Bedard">jbedard</assigned_to>
          <cc>ap</cc>
    
    <cc>commit-queue</cc>
    
    <cc>ews-watchlist</cc>
    
    <cc>glenn</cc>
    
    <cc>lforschler</cc>
    
    <cc>webkit-bug-importer</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1530546</commentid>
    <comment_count>0</comment_count>
    <who name="Jonathan Bedard">jbedard</who>
    <bug_when>2019-04-25 15:23:35 -0700</bug_when>
    <thetext>Lucas pointed out to me that upon completion of a layout test run, we will print out a description of how many tests passed vs the total number of tests. The problem with this metric is that the total number of tests includes the tests rightfully skipped because they are not run on the specified port.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1530547</commentid>
    <comment_count>1</comment_count>
    <who name="Jonathan Bedard">jbedard</who>
    <bug_when>2019-04-25 15:24:05 -0700</bug_when>
    <thetext>&lt;rdar://problem/49705424&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1530556</commentid>
    <comment_count>2</comment_count>
      <attachid>368280</attachid>
    <who name="Jonathan Bedard">jbedard</who>
    <bug_when>2019-04-25 15:55:03 -0700</bug_when>
    <thetext>Created attachment 368280
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1530584</commentid>
    <comment_count>3</comment_count>
      <attachid>368280</attachid>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2019-04-25 17:17:45 -0700</bug_when>
    <thetext>Comment on attachment 368280
Patch

Clearing flags on attachment: 368280

Committed r244671: &lt;https://trac.webkit.org/changeset/244671&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1530585</commentid>
    <comment_count>4</comment_count>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2019-04-25 17:17:46 -0700</bug_when>
    <thetext>All reviewed patches have been landed.  Closing bug.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>368280</attachid>
            <date>2019-04-25 15:55:03 -0700</date>
            <delta_ts>2019-04-25 17:17:45 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>bug-197298-20190425155502.patch</filename>
            <type>text/plain</type>
            <size>1370</size>
            <attacher name="Jonathan Bedard">jbedard</attacher>
            
              <data encoding="base64">SW5kZXg6IFRvb2xzL0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBUb29scy9DaGFuZ2VMb2cJKHJl
dmlzaW9uIDI0NDY2NykKKysrIFRvb2xzL0NoYW5nZUxvZwkod29ya2luZyBjb3B5KQpAQCAtMSwz
ICsxLDE0IEBACisyMDE5LTA0LTI1ICBKb25hdGhhbiBCZWRhcmQgIDxqYmVkYXJkQGFwcGxlLmNv
bT4KKworICAgICAgICB3ZWJraXRweTogSW5jbHVkaW5nIHNraXBwZWQgdGVzdHMgaW4gdGhlIGZp
bmFsIGNvdW50CisgICAgICAgIGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9p
ZD0xOTcyOTgKKyAgICAgICAgPHJkYXI6Ly9wcm9ibGVtLzQ5NzA1NDI0PgorCisgICAgICAgIFJl
dmlld2VkIGJ5IE5PQk9EWSAoT09QUyEpLgorCisgICAgICAgICogU2NyaXB0cy93ZWJraXRweS9s
YXlvdXRfdGVzdHMvdmlld3MvYnVpbGRib3RfcmVzdWx0cy5weToKKyAgICAgICAgKEJ1aWxkQm90
UHJpbnRlci5wcmludF9ydW5fcmVzdWx0cyk6IFRoZSB0b3RhbCBudW1iZXIgb2YgdGVzdHMgcnVu
IHNob3VsZCBleGNsdWRlIGRlbGlicmF0ZWx5IHNraXBwZWQgdGVzdHMuCisKIDIwMTktMDQtMjUg
IFRpbW90aHkgSGF0Y2hlciAgPHRpbW90aHlAYXBwbGUuY29tPgogCiAgICAgICAgIERpc2FibGUg
ZGF0ZSBhbmQgdGltZSBpbnB1dHMgb24gaU9TTWFjLgpJbmRleDogVG9vbHMvU2NyaXB0cy93ZWJr
aXRweS9sYXlvdXRfdGVzdHMvdmlld3MvYnVpbGRib3RfcmVzdWx0cy5weQo9PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0t
LSBUb29scy9TY3JpcHRzL3dlYmtpdHB5L2xheW91dF90ZXN0cy92aWV3cy9idWlsZGJvdF9yZXN1
bHRzLnB5CShyZXZpc2lvbiAyNDQ2NjMpCisrKyBUb29scy9TY3JpcHRzL3dlYmtpdHB5L2xheW91
dF90ZXN0cy92aWV3cy9idWlsZGJvdF9yZXN1bHRzLnB5CSh3b3JraW5nIGNvcHkpCkBAIC01Niw3
ICs1Niw3IEBAIGNsYXNzIEJ1aWxkQm90UHJpbnRlcihvYmplY3QpOgogCiAgICAgZGVmIHByaW50
X3J1bl9yZXN1bHRzKHNlbGYsIHJ1bl9yZXN1bHRzKToKICAgICAgICAgZmFpbGVkID0gcnVuX3Jl
c3VsdHMudG90YWxfZmFpbHVyZXMKLSAgICAgICAgdG90YWwgPSBydW5fcmVzdWx0cy50b3RhbAor
ICAgICAgICB0b3RhbCA9IHJ1bl9yZXN1bHRzLnRvdGFsIC0gcnVuX3Jlc3VsdHMuZXhwZWN0ZWRf
c2tpcHMKICAgICAgICAgcGFzc2VkID0gdG90YWwgLSBmYWlsZWQgLSBydW5fcmVzdWx0cy5yZW1h
aW5pbmcKICAgICAgICAgcGVyY2VudF9wYXNzZWQgPSAwLjAKICAgICAgICAgaWYgdG90YWwgPiAw
Ogo=
</data>

          </attachment>
      

    </bug>

</bugzilla>