Skip to content

webapp: fix broken log tail command#3229

Merged
yugangw-msft merged 2 commits intoAzure:masterfrom
yugangw-msft:logtail
May 8, 2017
Merged

webapp: fix broken log tail command#3229
yugangw-msft merged 2 commits intoAzure:masterfrom
yugangw-msft:logtail

Conversation

@yugangw-msft
Copy link
Copy Markdown
Contributor

@yugangw-msft yugangw-msft commented May 6, 2017

Fix #3227
This was broken by the latest functionapp integration. Add unit test coverage as functional test for log streaming is not applicable to run as an automation test

  • The PR has modified HISTORY.rst with an appropriate description of the change (see Modifying change log).

Command Guidelines

  • Each command and parameter has a meaningful description.
  • Each new command has a test.

(see Authoring Command Modules)

@codecov-io
Copy link
Copy Markdown

codecov-io commented May 6, 2017

Codecov Report

Merging #3229 into master will increase coverage by <.01%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #3229      +/-   ##
==========================================
+ Coverage   70.35%   70.35%   +<.01%     
==========================================
  Files         381      381              
  Lines       24777    24779       +2     
  Branches     3785     3786       +1     
==========================================
+ Hits        17431    17433       +2     
  Misses       6237     6237              
  Partials     1109     1109
Impacted Files Coverage Δ
...ice/azure/cli/command_modules/appservice/custom.py 73.54% <100%> (+0.06%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update bf2a6a3...75f4494. Read the comment docs.

@yugangw-msft yugangw-msft merged commit 4cb2e59 into Azure:master May 8, 2017
@yugangw-msft yugangw-msft deleted the logtail branch May 8, 2017 17:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants